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
herbstluftwm 0.8.3 -> 0.9.1 + add tests #104920
herbstluftwm 0.8.3 -> 0.9.1 + add tests #104920
Conversation
Result of 1 package built:
|
5f901a7
to
e0d66ca
Compare
e0d66ca
to
c2a1409
Compare
c2a1409
to
9c4280f
Compare
@ThibautMarty @SuperSandro2000 triage a rebuild? I would like to see this merged thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also please squash all herbstluftwm init related commits together.
9c4280f
to
84d858d
Compare
What are "herbstluftwm init" commits? This is not an "init package" PR, herbstluftwm was already packaged.
I'd rather prefer to not merge (3) and (4) (or even (5)), the distinction between the two can be useful when bisecting/cherry-picking/retrieving patchs/etc. But I will if you prefer. |
Converted to draft because 0.9.1 had been released. I will update the package in a few days. |
84d858d
to
90121cb
Compare
assert withDoc -> asciidoc != null; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
assert withDoc -> asciidoc != null; |
90121cb
to
190a3dc
Compare
I changed Please see my comment in the review for "asciidoc ? null" and the assertion. |
Could we, like, get this merged? |
No, because |
I'll fix that. We can wait #109455 to be merged to rebase onto. |
Also enables the packages' tests. The tests need a few patches to fix runtime generated script's shebangs. There is also a fix to pass PATH environment variable to subprocess' calls so that they can find some binaries (like xterm).
190a3dc
to
a6cdb11
Compare
Done. I removed the |
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 1 package built:
|
Motivation for this change
I checked that cross compilation still works (
crossSystem.config = "aarch64-unknown-linux-gnu"
).Replace #87003.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)