Skip to content

fix: failure to identify built packages on non-English locales#265

Merged
pbrisbin merged 4 commits into
archlinux-downgrade:mainfrom
Masterisk-F:fix-parse-nonEnglish
Feb 24, 2026
Merged

fix: failure to identify built packages on non-English locales#265
pbrisbin merged 4 commits into
archlinux-downgrade:mainfrom
Masterisk-F:fix-parse-nonEnglish

Conversation

@Masterisk-F

Copy link
Copy Markdown
Contributor

Checklist

  • Admin:
    • No duplicate PRs
  • Integration:
    • Always add unit tests for fixes or new functionality
    • If adding new options, update dist/completion/*
    • If adding new options, update dist/conf/downgrade.conf
  • Documentation:
    • If adding new options, update usage and doc/*.ronn
    • If adding new strings, update locale/*.po
    • If necessary, update README.md

Description

Currently, the script parses makepkg output log string "Finished making:" to identify built packages.
This approach fails when the output is localized (non-English locale).

This PR replaces build log parsing with makepkg --packagelist to identify built packages.

Comment thread src/downgrade Outdated
Corrections in response to suggestions
@pbrisbin
pbrisbin enabled auto-merge (rebase) February 20, 2026 15:38
Match "*makepkg -fs*" before "git*" so the mock correctly returns
MAKEPKG_FAIL for the combined "git checkout ... && makepkg -fs" command.
auto-merge was automatically disabled February 24, 2026 12:22

Head branch was pushed to by a user without write access

@Masterisk-F

Copy link
Copy Markdown
Contributor Author

Sorry for the late fix.
The test fix needed in #266 depends on the package retrieval method introduced here, so could you please merge this first?
I'll rebase #266 onto main afterward and fix the tests.

@pbrisbin
pbrisbin enabled auto-merge (rebase) February 24, 2026 13:18
@pbrisbin
pbrisbin merged commit 1e6fc55 into archlinux-downgrade:main Feb 24, 2026
3 checks passed
@pbrisbin

Copy link
Copy Markdown
Member

Done. Thanks!

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