[Bug 195645] Review Request: rasqal - RDF query library

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: rasqal - RDF query library


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





------- Additional Comments From thomas@xxxxxxxxxxxxx  2006-06-17 06:27 EST -------
Thanks for reviewing, much appreciated !

- changed license, but it still complains.  Filed
http://rpmlint.zarb.org/cgi-bin/trac.cgi/ticket/22

- re: changelog version; my name + email is rather long, and frequently extends
the width to more than 80 chars if I add version.  I filed 
http://rpmlint.zarb.org/cgi-bin/trac.cgi/ticket/23
and
http://rpmlint.zarb.org/cgi-bin/trac.cgi/ticket/24 (with patch)
If this is not critical I would like to keep this as-is for aesthetic reasons.

- got rid of rpath, ugh @ hacks.  This will teach me to actually lint on a 64
bit machine - apparently this doesn't cause trouble on a 32 bit one.

- %makeinstall: that wiki entry is factually wrong and lacking any actual
reasons except dogma - I will ask spot about it.  In the case of this package,
there is nothing wrong when doing makeinstall.  Unless you insist I prefer to
keep it.

- smp_mflags added
- OPTIMIZE removed
- make instead of %{__make} done

- make check added.  pinging upstream about test failures - since upstream did
not set it up to actually error out on those errors, and there are few, I'm
assuming for now these are expected or non-critical failures.

New version:
http://thomas.apestaart.org/download/pkg/fedora-5-i386-extras/rasqal-0.9.12-2.fc5/
Thanks again !


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]