Bug 743370
| Summary: | Review Request: libsmartpen - Library for communicating with the Livescribe Pulse Smartpen | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Nathanael Noblet <nathanael> |
| Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
| Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | mario.blaettermann, notting, package-review, tomspur |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2012-09-09 19:46:31 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | |||
| Bug Blocks: | 201449 | ||
|
Description
Nathanael Noblet
2011-10-04 17:41:04 UTC
Sorry...wrong url SRPM URL: http://www.gnat.ca/libsmartpen-0.6-1.fc15.src.rpm Where do you have python_sitelib0 from? Please use the macros from: fedoraproject.org/wiki/Packaging:Python Are you already a packager? I didn't found you in fas... I am a packager fas username is gnat. Also something I'm aware of but not sure how to solve yet. Upstream is in git and have released a few versions. However they don't build cleanly due to hardcoded paths in their Makefile. So I forked their git repo (github) and generated my own tarball that includes the patches I've made... How should this be packaged? take their tarball and using git history apply a bunch of patches until upstream includes them? Or just use my git fork pulling in new versions as they make them?? Spec URL: http://www.gnat.ca/libsmartpen.spec SRPM URL: http://www.gnat.ca/libsmartpen-0.6-2.src.rpm the sitelib0 was me copying from func.spec which has them conditionally defined but what I wanted was python_sitearch and didn't know they were provided by python-devel so just made up my own. I should have checked the guidelines instead of another spec I guess. (In reply to comment #3) > Upstream is in git and have released a few versions. However they don't build > cleanly due to hardcoded paths in their Makefile. So I forked their git repo > (github) and generated my own tarball that includes the patches I've made... > How should this be packaged? take their tarball and using git history apply a > bunch of patches until upstream includes them? Or just use my git fork pulling > in new versions as they make them?? You guess you need to use upstream tarball (or a git checkout) and do your patches on top of that. Having your own git repository only helps with upstreaming patches and keeping track of your changes. Using your repository as source would be a fork of upstream and not the real upstream source. (I think this would be a blocker) > Spec URL: http://www.gnat.ca/libsmartpen.spec > SRPM URL: http://www.gnat.ca/libsmartpen-0.6-2.src.rpm > > the sitelib0 was me copying from func.spec which has them conditionally defined > but what I wanted was python_sitearch and didn't know they were provided by > python-devel so just made up my own. I should have checked the guidelines > instead of another spec I guess. Thanks for changing it. Further comments: - BR and R of python is not needed. BR: python-devel is enought, the rest happens automatically. - please delete the --prefix on installing, or is it absolutely needed? - beeing a bit more expilcit in %files would be great, e.g. the subdirectories/files of %{python_sitearch} should be there, so python egg failures are detected. Updated: - Used upstream's tarball via github - Added patch between that tarball and my changes so it builds - more explicit %files - removed --prefix Spec URL: http://www.gnat.ca/libsmartpen.spec SRPM URL: http://www.gnat.ca/libsmartpen-0.6-3.src.rpm gosh... that srpm url is always wrong... Spec URL: http://www.gnat.ca/libsmartpen.spec SRPM URL: http://www.gnat.ca/libsmartpen-0.6-3.fc15.src.rpm Your spec and srpm links are dead again. Are you still working on this package? If not, we should close this review request (and also the depending bug #744083) and add it to FE-DEADREVIEW. Closing this review request, adding to FE-DEADREVIEW. |