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.

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

libpsl: added version 0.21.5 #23642

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

laci-aura
Copy link

Specify library name and version: libpsl/0.21.5


@conan-center-bot

This comment has been minimized.

@laci-aura laci-aura marked this pull request as ready for review April 18, 2024 10:41
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

Copy link
Member

@AbrilRBS AbrilRBS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot! I have a question regarding the new option, but therest looks good :)

@@ -76,7 +79,11 @@ def generate(self):
env.generate()
tc = MesonToolchain(self)
tc.project_options["runtime"] = self._idna_option
tc.project_options["builtin"] = self._idna_option
if Version(self.version) >= "0.21.5":
tc.project_options["builtin"] = self.options.enable_builtin
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this option is only valid from this version onward, it should be removed in the config_options() method so as to not influcence the pkgid of older versions :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, this PR that tried to target an older release https://github.com/conan-io/conan-center-index/pull/22029/files#diff-308e90de1d7646e4db40d401de640dd72698dbb96cb43871ac4fb7bbd8eb8566R77

has it still check for the old ._idna_option. I take that yours is more extensible, but could you confirm what the original upstream intention is? Thanks! :)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No it should not be removed, older versions of the lib used to allow setting the 'builtin' option to the same values as the 'runtime' option (libicu, libidn, etc) but now they changed the meaning/possible values to just True/False.
I preserved compatibility with older versions

tc.project_options["builtin"] = self._idna_option

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry took me a while to understand your comment :), fixed in 1e108a3

@AbrilRBS AbrilRBS self-assigned this Apr 18, 2024
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@laci-aura laci-aura requested a review from AbrilRBS April 20, 2024 10:36
@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ❌

Failure in build 8 (66f8191a711231ac07f2e44702e9c38720fab62f):

  • libpsl/0.21.5:
    CI failed to create some packages (All logs)

    Logs for packageID 890c7524504368b09d4192b22cba9f1160ef1009:
    [settings]
    arch=x86_64
    build_type=Release
    compiler=gcc
    compiler.libcxx=libstdc++11
    compiler.version=7
    os=Linux
    [options]
    libpsl:shared=False
    
    [...]
    -----------------
    [1/6] Compiling C object tools/psl.p/psl.c.o
    [2/6] Generating src/suffixes_dafsa.h with a custom command
    [3/6] Compiling C object src/libpsl.a.p/lookup_string_in_fixed_set.c.o
    [4/6] Compiling C object src/libpsl.a.p/psl.c.o
    [5/6] Linking static target src/libpsl.a
    [6/6] Linking target tools/psl
    INFO: autodetecting backend as ninja
    INFO: calculating backend command to run: /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/ninja/1.11.1/_/_/package/24647d9fe8ec489125dfbae4b3ebefaf7581674c/bin/ninja -j 3
    libpsl/0.21.5: Package '890c7524504368b09d4192b22cba9f1160ef1009' built
    libpsl/0.21.5: Build folder /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/build/890c7524504368b09d4192b22cba9f1160ef1009/build-release
    libpsl/0.21.5: Generated conaninfo.txt
    libpsl/0.21.5: Generated conanbuildinfo.txt
    libpsl/0.21.5: Generating the package
    libpsl/0.21.5: Package folder /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/package/890c7524504368b09d4192b22cba9f1160ef1009
    libpsl/0.21.5: Calling package()
    libpsl/0.21.5: Copied 1 file: COPYING
    libpsl/0.21.5: Copied 1 file: LICENSE
    
    ----Running------
    > . "/home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/build/890c7524504368b09d4192b22cba9f1160ef1009/build-release/conan/conanbuild.sh" && meson install -C "/home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/build/890c7524504368b09d4192b22cba9f1160ef1009/build-release" --destdir "/home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/package/890c7524504368b09d4192b22cba9f1160ef1009"
    -----------------
    ninja: Entering directory `/home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/build/890c7524504368b09d4192b22cba9f1160ef1009/build-release'
    ninja: no work to do.
    Installing src/libpsl.a to /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/package/890c7524504368b09d4192b22cba9f1160ef1009/lib
    Installing tools/psl to /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/package/890c7524504368b09d4192b22cba9f1160ef1009/bin
    Installing /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/build/890c7524504368b09d4192b22cba9f1160ef1009/build-release/include/libpsl.h to /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/package/890c7524504368b09d4192b22cba9f1160ef1009/include
    Installing /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/build/890c7524504368b09d4192b22cba9f1160ef1009/src/src/psl-make-dafsa.1 to /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/package/890c7524504368b09d4192b22cba9f1160ef1009/share/man/man1
    Installing /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/build/890c7524504368b09d4192b22cba9f1160ef1009/src/tools/psl.1 to /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/package/890c7524504368b09d4192b22cba9f1160ef1009/share/man/man1
    Installing /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/build/890c7524504368b09d4192b22cba9f1160ef1009/build-release/meson-private/libpsl.pc to /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/package/890c7524504368b09d4192b22cba9f1160ef1009/lib/pkgconfig
    Installing /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/build/890c7524504368b09d4192b22cba9f1160ef1009/src/src/psl-make-dafsa to /home/conan/workspace/prod-v1/bsr/46157/ffcea/.conan/data/libpsl/0.21.5/_/_/package/890c7524504368b09d4192b22cba9f1160ef1009/bin
    [HOOK - conan-center.py] post_package(): [PACKAGE LICENSE (KB-H012)] OK
    [HOOK - conan-center.py] post_package(): [DEFAULT PACKAGE LAYOUT (KB-H013)] If you are trying to package a tool put all the contents under the 'bin' folder
    [HOOK - conan-center.py] post_package(): [MATCHING CONFIGURATION (KB-H014)] OK
    [HOOK - conan-center.py] post_package(): [SHARED ARTIFACTS (KB-H015)] OK
    [HOOK - conan-center.py] post_package(): [STATIC ARTIFACTS (KB-H074)] OK
    [HOOK - conan-center.py] post_package(): [EITHER STATIC OR SHARED OF EACH LIB (KB-H076)] OK
    [HOOK - conan-center.py] post_package(): [PC-FILES (KB-H020)] OK
    [HOOK - conan-center.py] post_package(): [CMAKE-MODULES-CONFIG-FILES (KB-H016)] OK
    [HOOK - conan-center.py] post_package(): [PDB FILES NOT ALLOWED (KB-H017)] OK
    [HOOK - conan-center.py] post_package(): [LIBTOOL FILES PRESENCE (KB-H018)] OK
    [HOOK - conan-center.py] post_package(): [MS RUNTIME FILES (KB-H021)] OK
    [HOOK - conan-center.py] post_package(): [SHORT_PATHS USAGE (KB-H066)] OK
    [HOOK - conan-center.py] post_package(): [MISSING SYSTEM LIBS (KB-H043)] OK
    [HOOK - conan-center.py] post_package(): [APPLE RELOCATABLE SHARED LIBS (KB-H077)] OK
    
    Generating targets:   0%|          | 0/3 eta ?
                                                  
    
    Writing build.ninja:   0%|          | 0/32 eta ?
                                                    
    WARN: *** Conan 1 is legacy and on a deprecation path ***
    WARN: *** Please upgrade to Conan 2 ***
    libpsl/0.21.5: WARN: Using the new toolchains and generators without specifying a build profile (e.g: -pr:b=default) is discouraged and might cause failures and unexpected behavior
    [HOOK - conan-center.py] post_package(): ERROR: [DEFAULT PACKAGE LAYOUT (KB-H013)] Unknown folder 'share' in the package (https://github.com/conan-io/conan-center-index/blob/master/docs/error_knowledge_base.md#KB-H013-DEFAULT-PACKAGE-LAYOUT) 
    ERROR: [HOOK - conan-center.py] post_package(): Some checks failed running the hook, check the output
    
  • libpsl/0.21.1:
    Didn't run or was cancelled before finishing


Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants