Skip to content

zlib-ng: add v2.1.4#40647

Merged
michaelkuhn merged 1 commit intospack:developfrom
haampie:bump/zlib-ng-2
Oct 22, 2023
Merged

zlib-ng: add v2.1.4#40647
michaelkuhn merged 1 commit intospack:developfrom
haampie:bump/zlib-ng-2

Conversation

@haampie
Copy link
Copy Markdown
Member

@haampie haampie commented Oct 21, 2023

Let's drop the many patches in the next spack release

@michaelkuhn
Copy link
Copy Markdown
Member

Any reason for actually dropping the patches? In case someone wants to build 2.1.3 for archaeological reasons, it will fail, right?

@haampie
Copy link
Copy Markdown
Member Author

haampie commented Oct 22, 2023

Hm, I don't really see the point of having 2.1.3 + a thousand lines of patches, if it is more or less identical to 2.1.4.

@haampie
Copy link
Copy Markdown
Member Author

haampie commented Oct 22, 2023

Ok, I'll do it properly: keep the patches, deprecated the older patch releases, remove them + patches after Spack 0.21 release.

@haampie haampie changed the title zlib-ng: bump patch version, cleanup backports zlib-ng: add v2.1.4 Oct 22, 2023
Copy link
Copy Markdown
Member

@michaelkuhn michaelkuhn left a comment

Choose a reason for hiding this comment

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

LGTM

@michaelkuhn michaelkuhn enabled auto-merge (squash) October 22, 2023 16:25
@michaelkuhn michaelkuhn merged commit 7cd5fcb into spack:develop Oct 22, 2023
@haampie haampie deleted the bump/zlib-ng-2 branch October 22, 2023 17:39
mtaillefumier pushed a commit to mtaillefumier/spack that referenced this pull request Oct 23, 2023
victoria-cherkas pushed a commit to victoria-cherkas/spack that referenced this pull request Oct 30, 2023
RikkiButler20 pushed a commit to RikkiButler20/spack that referenced this pull request Nov 2, 2023
gabrielctn pushed a commit to gabrielctn/spack that referenced this pull request Nov 24, 2023
mtaillefumier pushed a commit to mtaillefumier/spack that referenced this pull request Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants