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
libow: init at 3.2p1 #59077
libow: init at 3.2p1 #59077
Conversation
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.
Please split your changes into two commits:
- Adding yourself as maintainer
- Adding the package
Done |
Now the commit where you add the package does not confirm to the CONTRIBUTING anymore. Sorry for bothering you with this, but as you seem to be new I try to show you how to do it right so that people do not have to tell you later 😄 |
Done ) |
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.
builds, links, on non-NixOS, no reverse-dependencies
New maintainer.
Previously approved by @matthiasbeyer
LGTM
This pull request has been mentioned on Nix community. There might be relevant details there: |
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.
@divi255 Thank you for your contribution! I'm sorry to hold this up just a tiny bit, but I noticed that at the top of the sourceforge page, there's a note: "Project moved to https://github.com/owfs/owfs/"
SourceForge as a distribution source for this package thus appears to be deprecated. Using the new source would allow us to also utilize the fetchFromGitHub
helper function to grab the newest release (v3.2p3) that is made available on GitHub as well.
Good day, I see but 3.2p1 is the only production release used by other distributions (the package was actually ported from Arch, while Ubuntu is still on 3.1.x). If you want I can change source to github right now, however I suggest to keep 3.2p1 version until the new one get a positive feedback (at least for x86-64 and ARM). Unfortunately I'm not a software author (but heavy user :) and can't give any warranties about new versions. So, let's change source to github right now or when versions newer than 3.2p1 go in production? What do you prefer? |
@divi255 Since you're way more of an expert than me on this piece of software, I defer to your opinion. Let's go with the github source using |
Changed to use github and fetchFromGitHub . Seems to work fine ) |
Thank you! |
Motivation for this change
libow 1-wire file system library (owfs) package
includes libraries and headers only
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)