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
bambootracker: init at 0.4.3 #81840
bambootracker: init at 0.4.3 #81840
Conversation
Seems to eval fine on x86 & ARM Linux, maybe a trusted user could kick off a Darwin build for completeness' sake? 🙂 |
f711544
to
ee08205
Compare
ee08205
to
f63f5b9
Compare
ed41722
to
6654ac3
Compare
6654ac3
to
14f8bbe
Compare
Updated to new v0.4.0 release. |
14f8bbe
to
00a956b
Compare
Noticed that I missed the i18n & skin files, added those. Does copying all the licensing files to |
I'd like to know that too. In my packages I didn't bother to include them if the makefile/whatever didn't do it by itself, but maybe I should have. |
6a20f2e
to
8c32baa
Compare
Updated to new v0.4.1 release. We added a workaround for the missing installation of i18n files on our side (it's caused by nixpkgs' qtbase version, QTBUG-77398), so I removed that part of the postInstall phase. |
8c32baa
to
649ec45
Compare
Bumped again to v0.4.2. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review-may-2019/3032/175 |
649ec45
to
3e62bca
Compare
Bumped to v0.4.3. |
@GrahamcOfBorg build bambootracker |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Let's see if this attracts some committer :) |
I added a patch to fix the Darwin build, it can be dropped with the next release. Additionally, I ran |
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.
Jikes, I am sorry that merging this took so long! I added some small comments so that the derivation follows the guidelines. It would also be nice if you could squash the two commits into one.
Feel free to ping me when you have incorporated these changes, so that we can get this merged!
postPatch = "cd BambooTracker"; | ||
|
||
# sourceRoot = "source/BambooTracker"; |
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.
Any reason sourceRoot
is not used?
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.
Makes patch from GitHub commit fail.
patching sources
applying patch /nix/store/05w1rpcw9j4y1wik5bqblhc4vnzhmdkv-45346ed99559d44c2e32a5c6138a0835b212e875.patch
can't find file to patch at input line 3
515eca9
to
fb6fe82
Compare
fb6fe82
to
0e9f384
Compare
Can't use |
Am I doing something wrong btw? GitHub CI is flooding me with "Run failed" emails. |
Thanks for making the changes! Not sure where the GitHub CI errors are coming from. But ofborg is happy. |
Thanks for merging! 👍 |
Motivation for this change
Been in my local checkout for awhile, decided to push since it was mentioned in #81815.
Ping @fgaz because you were interested in packaging it. Give it a look, let me know how it looks to you.
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) (new package -> 326780408)