[Bug 1562526] Review Request: libfullock - A Fast User Level LOCK (FULLOCK ) library for C and C++

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1562526



--- Comment #7 from Hiro Wakabayashi <hiwkby@xxxxxxxxx> ---
Hello, Iwicki

I would like to ask you to continue this review as it is if it is not a big
problem because upstream currently declined my idea for a reasonable reason.

I discussed about my idea with the upstream developer yesterday, but he said 
it was difficult to apply it because he wants to get its program to 
automatically print out its git commit hash to track down which version of 
the software a user is using for user support.

I left notes in comments above the Source0: line to make tarball as mentioned
in the following guideline though I still should continue to think about 
another idea to make the url not to be one of "Troublesome URLs".
https://fedoraproject.org/wiki/Packaging:SourceURL

Thanks in advance.

Memos:
Code using git rev-parse command is here.
https://github.com/yahoojapan/fullock/blob/master/make_rev.sh#L34
https://github.com/yahoojapan/fullock/blob/master/lib/Makefile.am#L41

But tarball from the following url doesn't contains .git directory.
https://github.com/yahoojapan/fullock/archive/v1.0.24.tar.gz

As a result, git rev-parse exit with nonzero status.
https://git-scm.com/docs/git-rev-parse

So I commented how to generate the tarball.
https://github.com/hiwkby/fullock/blob/master/libfullock.spec#L10-L13

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux