Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Url varexpand issues #497

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from
Draft

Url varexpand issues #497

wants to merge 9 commits into from

Commits on Dec 15, 2023

  1. fix typo in comment

    @mlandres
    mlandres committed Dec 15, 2023
    Configuration menu
    Copy the full SHA
    3d8667d View commit details
    Browse the repository at this point in the history

  2. Url: Fix bug when parsing empty components like query or fragment 

    As a regex is used anyway there's also no need for substring actions.
    @mlandres
    mlandres committed Dec 15, 2023
    Configuration menu
    Copy the full SHA
    cf3d9ab View commit details
    Browse the repository at this point in the history

  3. Url: Fix regexes; '.' inside a character class must not be \-escaped

    @mlandres
    mlandres committed Dec 15, 2023
    Configuration menu
    Copy the full SHA
    9d4853e View commit details
    Browse the repository at this point in the history

  4. RepoVarExpand: Fix expansion of embedded words with invalid variable … 

    …definitions

An incomplete variable definition is to be expanded literally. It should
produce the same result as if the the introducing '$' was escaped.
    @mlandres
    mlandres committed Dec 15, 2023
    Configuration menu
    Copy the full SHA
    9ce8905 View commit details
    Browse the repository at this point in the history

  5. RepoVarExpand: VarRetriever callback is not needed for backslash esca… 

    …ped literals
    @mlandres
    mlandres committed Dec 15, 2023
    Configuration menu
    Copy the full SHA
    1faba2a View commit details
    Browse the repository at this point in the history

  6. Add hasRepoVarsEmbedded test for strings

    @mlandres
    mlandres committed Dec 15, 2023
    Configuration menu
    Copy the full SHA
    82d408f View commit details
    Browse the repository at this point in the history

  7. Url: Add builtin "zypp-rawurl" schema for strings with unexpanded rep… 

    …o variables

The schema is used internally to wrap raw strings with unexpanded
repo variables into type Url. The raw string is stored in the Url's
fragment because it does not necessarily form a valid Url before it
got expanded.
    @mlandres
    mlandres committed Dec 15, 2023
    Configuration menu
    Copy the full SHA
    3219d97 View commit details
    Browse the repository at this point in the history

  8. Add RawUrl for constructing the zypp-rawurl: schema and handle it in … 

    …RepoVariablesUrlReplacer
    @mlandres
    mlandres committed Dec 15, 2023
    Configuration menu
    Copy the full SHA
    b889e22 View commit details
    Browse the repository at this point in the history

  9. WIP 

    Repo/Service file: Full support for using repo variables in URLs (bsc#1217192)

With a libzypp providing 'libzypp(repovarexpand) >= 1.2' the use of
repo variables in URLs is no longer restricted to the host and path
component of an URL, nor must the not-expanded string form a valid URL.
    @mlandres
    mlandres committed Dec 15, 2023
    Configuration menu
    Copy the full SHA
    a7d1a12 View commit details
    Browse the repository at this point in the history