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
Timekeeper gtk app #102265
Timekeeper gtk app #102265
Conversation
had to make some minor adjustments to the suggestions - parens and the mkdir failed without the -p. |
Can you squash the commits together? The repo has so many so we want to keep not to many and it makes backporting way easier. Also you don't need to apply everything I suggest. I just do a quick look for nitpicks. |
6b8d2a5
to
3f61fbe
Compare
poco | ||
pkgconfig | ||
] ++ (with gnome2; [ | ||
gtkmm |
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, is it not possible to build the app against gtkmm3
? GTK 2 and libglade has been deprecated for ages.
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.
I haven't worked on this code in about 10 years. I wanted to just compile it and make it available, not rewrite it. I'll see if it will work without having to work on the code. If I have to do a significant rewrite I'll probably bail on publishing.
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.
Ok I fixed all the things but not this. If its really a deal breaker I'll see what I can do.
Finally, could you please follow the contribution guide for commit messages: https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes |
Result of 1 package built:
|
Motivation for this change
Add my TimeKeeper gtk app for tracking hours.
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)