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

Set correct pull request message for --dry-run case #1

Open
wants to merge 25 commits into
base: nixos-support
Choose a base branch
from

Conversation

robwoolley
Copy link
Owner

When --only is used, if --dry-run is used it calls save_pr() otherwise file_pr() is called. This ensures that the same string value is passed to the delta parameter of both functions.

lopsided98 and others added 25 commits March 28, 2022 19:49
This makes Nix package generation nearly 4 times faster, if tarballs are already
cached. Some of the optimizations also apply to other distros, but the
performance impact there was not tested.
GitLab returns a 403 error if the default urllib user agent is used.
This allows conditions in package.xml to be evaluated correctly. Mostly
copied from the Bitbake generator.
This is required to make strictDeps/cross-compilation work.
This name better represents what this class does, which is to generate the
actual text of a Nix expression for a package.
The license translation logic has changed, and adds a hyphen to unknown
licenses. Also, add a test for the public domain license.
When --all is specified, generate for rolling as well.
The license string in the generated nix package file was broken when the
license name contained quote.

It has been fixed. Backslash and the sequence "dollar opening bracket"
have been escaped too.
Use the same function to escape the description string as is used for
the license string.
When --only is used, if --dry-run is used it calls save_pr() otherwise
file_pr() is called. This ensures that the same string value is passed
to the delta parameter of both functions.

Signed-off-by: Rob Woolley <[email protected]>
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