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 ncurses to latest. #9128

Merged
merged 2 commits into from
Nov 28, 2023
Merged

Update ncurses to latest. #9128

merged 2 commits into from
Nov 28, 2023

Conversation

vaikas
Copy link
Member

@vaikas vaikas commented Nov 28, 2023

While working on the package tests for melange, noticed the ncurses was failing:
https://github.com/chainguard-dev/melange/actions/runs/7008726917/job/19065682639?pr=864

⚠️  x86_64    | --2023-11-27 17:40:51--  https://invisible-mirror.net/archives/ncurses/current/ncurses-6.4-20230722.tgz
⚠️  x86_64    | Resolving invisible-mirror.net... 160.153.42.69
⚠️  x86_64    | Connecting to invisible-mirror.net|160.153.42.69|:443... connected.
⚠️  x86_64    | HTTP request sent, awaiting response... 404 Not Found

And sure enough, it's not there anymore. So bump the later one.

Fixes:

Related:

Pre-review Checklist

For new package PRs only

  • This PR is marked as fixing a pre-existing package request bug
    • Alternatively, the PR is marked as related to a pre-existing package request bug, such as a dependency
  • REQUIRED - The package is available under an OSI-approved or FSF-approved license
  • REQUIRED - The version of the package is still receiving security updates
  • This PR links to the upstream project's support policy (e.g. endoflife.date)

For new version streams

  • The upstream project actually supports multiple concurrent versions.
  • Any subpackages include the version string in their package name (e.g. name: ${{package.name}}-compat)
  • The package (and subpackages) provides: logical unversioned forms of the package (e.g. nodejs, nodejs-lts)

For security-related PRs

  • The security fix is recorded in the advisories repo

For version bump PRs

  • The epoch field is reset to 0

For PRs that add patches

  • Patch source is documented

Signed-off-by: Ville Aikas <[email protected]>
@vaikas
Copy link
Member Author

vaikas commented Nov 28, 2023

Bleh, I just noticed #9086 that seems to tackle this.

Copy link
Member

@hectorj2f hectorj2f left a comment

Choose a reason for hiding this comment

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

Don't you need to update the epoch?

Signed-off-by: Ville Aikas <[email protected]>
@vaikas
Copy link
Member Author

vaikas commented Nov 28, 2023

Don't you need to update the epoch?

doh, I need to drop it back to 0 since version increased.

@vaikas vaikas enabled auto-merge November 28, 2023 09:07
@vaikas vaikas added this pull request to the merge queue Nov 28, 2023
Merged via the queue into wolfi-dev:main with commit 405cb36 Nov 28, 2023
@vaikas vaikas deleted the ncurses-bump branch November 28, 2023 09:15
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.

2 participants