[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 Jason Tibbitts changed: What|Removed |Added Status|ASSIGNED|NEW Blocks|201449(FE-DEADREVIEW) | --- Comment #17 from Jason Tibbitts 2011-06-08 16:13:55 EDT --- The only direct URLs to the spec and srpm (in the original comment) are invalid. Please post correct and direct URLs to the spec and srpm. Also, you neglected to unblock FE-DEADREVIEW. I cleared that out for you. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 Fabian Deutsch changed: What|Removed |Added Status|CLOSED |ASSIGNED Resolution|NOTABUG | Keywords||Reopened --- Comment #16 from Fabian Deutsch 2011-05-17 07:50:39 EDT --- Upstream finally resolved the conflict. I am reopening this ticket. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 Jason Tibbitts changed: What|Removed |Added Status|NEW |CLOSED Blocks||201449(FE-DEADREVIEW) Resolution||NOTABUG Status Whiteboard|StalledSubmitter| Last Closed||2010-12-17 12:50:47 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 Jason Tibbitts changed: What|Removed |Added Status Whiteboard||StalledSubmitter -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 --- Comment #15 from Fabian Deutsch 2010-07-02 05:49:28 EDT --- I'm currently quite bussy at the moment. When I get a bit of time, I will take the points above to resolve the current Conflict with libdnet. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 --- Comment #14 from Martin Gieseking 2010-06-21 10:05:28 EDT --- (In reply to comment #13) > I hope the library function names don't clash with those in libdnet, since > otherwise also this is asking for trouble. As far as I see, there are no symbol naming conflicts. > Upstream should rename the tarball to 'elliptics'. However, until that happens > this review has the wrong name, as per Fedora naming guidelines. The correct > package name would be, of course, dnet. If I understand the naming guidelines correctly, the package name could also be elliptics, since it's the name of the project: "When naming a package, the name should match the upstream tarball or project name from which this software came." But maybe I just didn't get the intended meaning. However, I find it pretty strange to call the project "elliptics" and use "dnet" everywhere else. So, dnet is perhaps indeed the better package name. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=569671
Jussi Lehtola changed:
What|Removed |Added
CC||jussi.leht...@iki.fi
--- Comment #13 from Jussi Lehtola 2010-06-21 08:34:37
EDT ---
(In reply to comment #12)
> Hi Fabian,
>
> since using Conflicts should be avoided in Fedora packages, I suggest to move
> the library (and its symlinks) to a subdirectory of %{_libdir}, e.g.
> %{_libdir}/elliptics/, as long as the conflict persists. In this case, you
> also
> have to add a file elliptics-%{_arch}.conf to /etc/ld.so.conf.d containing the
> directory where the library is located.
> Nonetheless, it would be much better if upstream renamed the library.
I hope the library function names don't clash with those in libdnet, since
otherwise also this is asking for trouble.
Upstream should rename the tarball to 'elliptics'. However, until that happens
this review has the wrong name, as per Fedora naming guidelines. The correct
package name would be, of course, dnet.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=569671
--- Comment #12 from Martin Gieseking 2010-06-21
07:55:25 EDT ---
Hi Fabian,
since using Conflicts should be avoided in Fedora packages, I suggest to move
the library (and its symlinks) to a subdirectory of %{_libdir}, e.g.
%{_libdir}/elliptics/, as long as the conflict persists. In this case, you also
have to add a file elliptics-%{_arch}.conf to /etc/ld.so.conf.d containing the
directory where the library is located.
Nonetheless, it would be much better if upstream renamed the library.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 --- Comment #11 from Fabian Deutsch 2010-06-19 12:32:24 EDT --- Thanks Martin. I applied those changes. Spec and srpm can be found at http://fedorapeople.org/~fabiand/elliptics/ A quick rpmlint on local files just shows the usual warnings. Koji build was also fine. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=569671
--- Comment #10 from Martin Gieseking 2010-06-17
15:25:43 EDT ---
Here are a couple of quick comments:
- in the first line, replace %define with %global
- according to the source file headers, the license is GPLv2+, not GPLv2
- I suggest to use the default %description for the devel package:
The %{name}-devel package contains libraries and header files for
developing applications that use %{name}.
- remove the duplicate %defattr line in %files devel
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 --- Comment #9 from Fabian Deutsch 2010-06-16 08:15:46 EDT --- So, I upted the spec ad fixed two problems: - Removed DSO fix (fiexd upstream) - Removed rpath for x86_64 rpmlint just throws a couple of warning about non existant man-pages (see below). SPEC and SRPM: http://fedorapeople.org/~fabiand/elliptics/ Koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=2253194 rpmlint: $ rpmlint -v elliptics-* elliptics.i686: I: checking elliptics.i686: W: spelling-error %description -l en_US dnet -> dent, net, diet elliptics.i686: W: spelling-error %description -l en_US ioserver -> observer, timeserver, listserver elliptics.i686: I: checking-url http://www.ioremap.net/projects/elliptics/ (timeout 10 seconds) elliptics.i686: W: no-manual-page-for-binary dnet_fcgi elliptics.i686: W: no-manual-page-for-binary dnet_check_merge elliptics.i686: W: no-manual-page-for-binary dnet_check_create elliptics.i686: W: no-manual-page-for-binary dnet_stat elliptics.i686: W: no-manual-page-for-binary dnet_iotest elliptics.i686: W: no-manual-page-for-binary dnet_check_log elliptics.i686: W: no-manual-page-for-binary dnet_notify elliptics.i686: W: no-manual-page-for-binary dnet_hparser elliptics.i686: W: no-manual-page-for-binary dnet_ioserv elliptics.src: I: checking elliptics.src: W: spelling-error %description -l en_US dnet -> dent, net, diet elliptics.src: W: spelling-error %description -l en_US ioserver -> observer, timeserver, listserver elliptics.src: I: checking-url http://www.ioremap.net/projects/elliptics/ (timeout 10 seconds) elliptics.src: I: checking-url http://www.ioremap.net/archive/elliptics/dnet-2.8.0.tar.gz (timeout 10 seconds) elliptics.x86_64: I: checking elliptics.x86_64: W: spelling-error %description -l en_US dnet -> dent, net, diet elliptics.x86_64: W: spelling-error %description -l en_US ioserver -> observer, timeserver, listserver elliptics.x86_64: I: checking-url http://www.ioremap.net/projects/elliptics/ (timeout 10 seconds) elliptics.x86_64: W: no-manual-page-for-binary dnet_fcgi elliptics.x86_64: W: no-manual-page-for-binary dnet_check_merge elliptics.x86_64: W: no-manual-page-for-binary dnet_check_create elliptics.x86_64: W: no-manual-page-for-binary dnet_stat elliptics.x86_64: W: no-manual-page-for-binary dnet_iotest elliptics.x86_64: W: no-manual-page-for-binary dnet_check_log elliptics.x86_64: W: no-manual-page-for-binary dnet_notify elliptics.x86_64: W: no-manual-page-for-binary dnet_hparser elliptics.x86_64: W: no-manual-page-for-binary dnet_ioserv elliptics-debuginfo.i686: I: checking elliptics-debuginfo.i686: I: checking-url http://www.ioremap.net/projects/elliptics/ (timeout 10 seconds) elliptics-debuginfo.x86_64: I: checking elliptics-debuginfo.x86_64: I: checking-url http://www.ioremap.net/projects/elliptics/ (timeout 10 seconds) elliptics-devel.i686: I: checking elliptics-devel.i686: I: checking-url http://www.ioremap.net/projects/elliptics/ (timeout 10 seconds) elliptics-devel.x86_64: I: checking elliptics-devel.x86_64: I: checking-url http://www.ioremap.net/projects/elliptics/ (timeout 10 seconds) 7 packages and 0 specfiles checked; 0 errors, 24 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 --- Comment #8 from Martin Gieseking 2010-06-15 03:42:23 EDT --- OK, that's fine. Thanks for the info. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 --- Comment #7 from Fabian Deutsch 2010-06-15 03:38:24 EDT --- Hi, yes I am. I waited for upstream to change it's name, to solve the naming conflict with libdnet. As this did not happen I'm planning to add a Conflicts tag to the spec. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 Martin Gieseking changed: What|Removed |Added CC||martin.giesek...@uos.de --- Comment #6 from Martin Gieseking 2010-06-15 03:30:53 EDT --- Hi Fabian, are you still interested in this package? In the meantime, upstream released several updates. Maybe you'd like to update the package, too. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 --- Comment #5 from Fabian Deutsch 2010-03-02 17:07:15 EST --- Upstream has been contacted, is open about changing the name. No conflict appeard up to now, because the conflicting package libdnet wasn't present. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 --- Comment #4 from Mamoru Tasaka 2010-03-02 01:50:26 EST --- First contacting the upstream is better. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 --- Comment #3 from Fabian Deutsch 2010-03-01 20:39:08 EST --- Oh. When thinking about this: There are naming conflicts. libdnet provides libdnet.so as elliptics does to. What can be done? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 --- Comment #1 from Fabian Deutsch 2010-03-01 20:36:36 EST --- [fabi...@proprietary rpmbuild]$ rpmlint -v SPECS/elliptics.spec SRPMS/elliptics-2.6.4-1.fc14.src.rpm RPMS/i686/elliptics-* SPECS/elliptics.spec: I: checking-url http://www.ioremap.net/archive/elliptics/dnet-2.6.4.tar.gz (timeout 10 seconds) elliptics.src: I: checking elliptics.src: W: spelling-error %description -l en_US dnet -> dent, net, diet elliptics.src: W: spelling-error %description -l en_US ioserver -> observer, timeserver, servery elliptics.src: I: checking-url http://www.ioremap.net/projects/elliptics/ (timeout 10 seconds) elliptics.src: I: checking-url http://www.ioremap.net/archive/elliptics/dnet-2.6.4.tar.gz (timeout 10 seconds) elliptics.i686: I: checking elliptics.i686: W: spelling-error %description -l en_US dnet -> dent, net, diet elliptics.i686: W: spelling-error %description -l en_US ioserver -> observer, timeserver, servery elliptics.i686: I: checking-url http://www.ioremap.net/projects/elliptics/ (timeout 10 seconds) elliptics-debuginfo.i686: I: checking elliptics-debuginfo.i686: I: checking-url http://www.ioremap.net/projects/elliptics/ (timeout 10 seconds) elliptics-devel.i686: I: checking elliptics-devel.i686: I: checking-url http://www.ioremap.net/projects/elliptics/ (timeout 10 seconds) 4 packages and 1 specfiles checked; 0 errors, 4 warnings. spelling-error: Nope. dnet is correct. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569671] Review Request: elliptics - Distributed hash table storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569671 --- Comment #2 from Fabian Deutsch 2010-03-01 20:38:11 EST --- What might be a bit confusing is the naming of the project and package. I chose to use elliptics as it is popular upstream and dnet is already taken (libdnet). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
