Skip to content

Conversation

@joshrwolf
Copy link
Member

@joshrwolf joshrwolf commented Jul 6, 2023

update to the new 20230701 source for ncurses

if there's a more accepted way of pinning to a source for ncurses, please let me know :|

part of #3580

@joshrwolf joshrwolf requested a review from a team as a code owner July 6, 2023 14:05
@joshrwolf joshrwolf force-pushed the ncurses-20230701 branch 2 times, most recently from 06b7edb to 5e1d727 Compare July 6, 2023 14:13
jdolitsky
jdolitsky previously approved these changes Jul 6, 2023
@jdolitsky
Copy link
Member

2023/07/06 14:14:15 attempting to bump version to 6.4-20230701
2023/07/06 14:14:15 processing fetch node:
2023/07/06 14:14:15   uri: [https://distfiles.alpinelinux.org/distfiles/edge/ncurses-${{package.version}}-20230701.tgz](https://distfiles.alpinelinux.org/distfiles/edge/ncurses-$%7B%7Bpackage.version%7D%7D-20230701.tgz)
2023/07/06 14:14:15   evaluated: https://distfiles.alpinelinux.org/distfiles/edge/ncurses-6.4-20230701-20230701.tgz
time="2023-07-06T14:14:15Z" level=fatal msg="error during command execution: 1 error occurred:\n\t* package ncurses: failed to validate update config, melange bump: got 404 Not Found when fetching [https://distfiles.alpinelinux.org/distfiles/edge/ncurses-6.4-20230701-20230701.tgz\n\n](https://distfiles.alpinelinux.org/distfiles/edge/ncurses-6.4-20230701-20230701.tgz/n/n)"

but manual: true is set... not sure

@joshrwolf
Copy link
Member Author

going to hold off on this till @kaniini gets a chance to weigh in

@jdolitsky jdolitsky added this pull request to the merge queue Jul 6, 2023
@jdolitsky jdolitsky removed this pull request from the merge queue due to a manual request Jul 6, 2023
ncurses.yaml Outdated
@@ -1,7 +1,7 @@
package:
name: ncurses
version: 6.4
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we should do something like

Suggested change
version: 6.4
version: 6.4_p20230701

Copy link
Member Author

Choose a reason for hiding this comment

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

updated, still can't auto update with wolfictl, but at least it's clearer now when we're bumping the source

does this need a provides ncurses=6.4? or is it smart enough to figure that out

@kaniini
Copy link
Contributor

kaniini commented Jul 25, 2023

Superceded by #3867.

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.

3 participants