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

Update 3.7.7 #13

Merged
merged 10 commits into from
Dec 11, 2024
Merged

Update 3.7.7 #13

merged 10 commits into from
Dec 11, 2024

Conversation

Arishamays1
Copy link

@Arishamays1 Arishamays1 commented Nov 13, 2024

☆Libarchive 3.7.7☆

Jira Ticket
Upstream

Changes

  • Updated version number and sha256
  • Updated dependencies for run and host
  • Removed outdated patches

@Arishamays1 Arishamays1 self-assigned this Nov 13, 2024
@anaconda-pkg-build
Copy link

Linter check found the following problems: ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_59d_go0fhv/clone` failed. (See above for error) The following problems have been found:

===== ERRORS =====

  • clone/recipe/meta.yaml:27: patch_unnecessary: patch should not be in build when source/patches is not set
    ===== Final Report: =====
    1 Error and 0 Warnings were found

@anaconda-pkg-build
Copy link

Linter check found the following problems: ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_9883msne79/clone` failed. (See above for error) The following problems have been found:

===== ERRORS =====

  • clone/recipe/meta.yaml:18: patch_unnecessary: patch should not be in build when source/patches is not set
    ===== Final Report: =====
    1 Error and 0 Warnings were found

@anaconda-pkg-build
Copy link

Linter check found the following problems: Traceback (most recent call last): File "/opt/conda/envs/linter/lib/python3.12/site-packages/anaconda_linter/run.py", line 208, in main return_code, report = execute_linter( ^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/anaconda_linter/run.py", line 180, in execute_linter result = linter.lint(recipes, subdir, variant_config_files, exclusive_config_files, fix_flag) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/anaconda_linter/lint/__init__.py", line 725, in lint msgs = self.lint_file( ^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/anaconda_linter/lint/__init__.py", line 823, in lint_file recipe = _recipe.Recipe.from_file( ^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/percy/render/recipe.py", line 243, in from_file raise exc from exc File "/opt/conda/envs/linter/lib/python3.12/site-packages/percy/render/recipe.py", line 234, in from_file recipe._load_from_string(text.read()) File "/opt/conda/envs/linter/lib/python3.12/site-packages/percy/render/recipe.py", line 170, in _load_from_string self.render() File "/opt/conda/envs/linter/lib/python3.12/site-packages/percy/render/recipe.py", line 290, in render self.meta = renderer_utils.render(self.recipe_dir, self.dump(), self.selector_dict, self.renderer) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/percy/render/_renderer.py", line 245, in render return ruamel.load(yaml_text.replace("\t", " ").replace("%", " ")) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/main.py", line 451, in load return constructor.get_single_data() ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/constructor.py", line 114, in get_single_data node = self.composer.get_single_node() ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/composer.py", line 72, in get_single_node document = self.compose_document() ^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/composer.py", line 94, in compose_document node = self.compose_node(None, None) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/composer.py", line 130, in compose_node node = self.compose_mapping_node(anchor) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/composer.py", line 211, in compose_mapping_node item_value = self.compose_node(node, item_key) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/composer.py", line 130, in compose_node node = self.compose_mapping_node(anchor) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/composer.py", line 211, in compose_mapping_node item_value = self.compose_node(node, item_key) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/composer.py", line 128, in compose_node node = self.compose_sequence_node(anchor) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/composer.py", line 172, in compose_sequence_node while not self.parser.check_event(SequenceEndEvent): ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/parser.py", line 141, in check_event self.current_event = self.state() ^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/parser.py", line 535, in parse_block_sequence_entry if self.scanner.check_token(BlockEntryToken): ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/scanner.py", line 1733, in check_token while self.need_more_tokens(): ^^^^^^^^^^^^^^^^^^^^^^^ File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/scanner.py", line 203, in need_more_tokens self.stale_possible_simple_keys() File "/opt/conda/envs/linter/lib/python3.12/site-packages/ruamel/yaml/scanner.py", line 348, in stale_possible_simple_keys raise ScannerError( ruamel.yaml.scanner.ScannerError: while scanning a simple key in "", line 20, column 1: requirements:; ^ (line: 20) could not find expected ':' in "", line 21, column 8: build: ^ (line: 21)

ERROR conda.cli.main_run:execute(125): conda run conda-lint /tmp/abs_4embbf5mul/clone failed. (See above for error)

@anaconda-pkg-build
Copy link

Linter check found the following problems: ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_9e62zzbkhp/clone` failed. (See above for error) The following problems have been found:

===== ERRORS =====

  • clone/recipe/meta.yaml:18: patch_unnecessary: patch should not be in build when source/patches is not set
    ===== Final Report: =====
    1 Error and 0 Warnings were found

@cbouss cbouss self-requested a review November 14, 2024 21:01
@cbouss
Copy link

cbouss commented Nov 14, 2024

Can you revert the changes to the patches? It seems line endings have been modified.

@cbouss
Copy link

cbouss commented Nov 14, 2024

I just saw this ticket related: https://anaconda.atlassian.net/browse/PKG-5733
Looking at the content of the produced package, the soname indeed changed from libarchive.so.20 in 3.7.4 to libarchive.so.13 in 3.7.7. Let's pause this for now.

@cbouss
Copy link

cbouss commented Nov 14, 2024

Or we could undo libarchive/libarchive#1976 for this update. And wait for a 3.8 release to switch to the new soname.

@Arishamays1
Copy link
Author

I just saw this ticket related: https://anaconda.atlassian.net/browse/PKG-5733 Looking at the content of the produced package, the soname indeed changed from libarchive.so.20 in 3.7.4 to libarchive.so.13 in 3.7.7. Let's pause this for now.

This is fine! I can put this on hold for now and follow up on it if that works!

@JeanChristopheMorinPerso

@cbouss however painful the ABI break might be, I think we need to do it now. I don't know when 3.8 will come out and it might not even happen.

This ABI breakage is a big problem that causes a bunch of issues between us and conda-forge which in itself breaks conda (or more libmamba, which breas conda-libmamba-solver which breaks conda).

I really think we should align with conda-forge here.

@cbouss
Copy link

cbouss commented Dec 10, 2024

ABI breakage addressed by AnacondaRecipes/repodata-hotfixes#244

@cbouss cbouss merged commit c9e53b0 into master Dec 11, 2024
8 checks passed
@cbouss cbouss deleted the update-3.7.7 branch December 11, 2024 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants