Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
Summary: Review Request: XaraLX - Vector/general purpose graphics utility Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: bugzilla-sink@leemhuis.info ReportedBy: tcallawa@redhat.com QAContact: fedora-package-review@redhat.com
Spec URL: http://www.auroralinux.org/people/spot/review/XaraLX.spec SRPM URL: http://www.auroralinux.org/people/spot/review/XaraLX-0.5-1.src.rpm Description: General purpose and vector graphics program, supports photo, graphics, and illustration creation and editing.
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
gemi@bluewin.ch changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |gemi@bluewin.ch
------- Additional Comments From gemi@bluewin.ch 2006-05-27 11:34 EST ------- Does not build in mock: "autoreconf: failed to run autopoint: No such file or directory" Needs BuildReq: gettext-devel
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
------- Additional Comments From rc040203@freenet.de 2006-05-28 00:38 EST ------- (In reply to comment #1)
Does not build in mock: "autoreconf: failed to run autopoint: No such file or directory" Needs BuildReq: gettext-devel
You should not run autoreconf in specs - You should use patches.
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
------- Additional Comments From gemi@bluewin.ch 2006-07-26 11:41 EST ------- The latest version 0.7 rev 1537 builds fine here.
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
paul@all-the-johnsons.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |paul@all-the-johnsons.co.uk
------- Additional Comments From paul@all-the-johnsons.co.uk 2006-07-26 17:42 EST ------- #3 - have you got a spec file you can upload somewhere?
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
------- Additional Comments From gemi@bluewin.ch 2006-07-26 18:17 EST ------- (In reply to comment #4)
#3 - have you got a spec file you can upload somewhere?
I simply used the one from comment #1. I suspect that spot is not very interesting in this package anymore or is to busy with other things currently. Maybe someone else can step in?
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
------- Additional Comments From paul@all-the-johnsons.co.uk 2006-07-27 02:23 EST ------- #5 - someone has. me.
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
------- Additional Comments From paul@all-the-johnsons.co.uk 2006-07-27 08:56 EST ------- Spec URL: http://www.knox.net.nz/~nodoid/XaraLX.spec SRPM URL: http://www.knox.net.nz/~nodoid/XaraLX-0.7-1.r1564.src.rpm
- bump to new version - includes 64 bit fixes - Additional docs - fixed EOL for xaralx.htm - fixed multiple mode errors for header and source files - removed mixed use of spaces and tabs
The package builds cleanly on x86_64 with rpmlint not giving a single complaint! I've not built it in mock yet (my testrig is not working happily today from a remote login).
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
michael@knox.net.nz changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|bugzilla-sink@leemhuis.info |michael@knox.net.nz OtherBugsDependingO|163776 |163778 nThis| |
------- Additional Comments From michael@knox.net.nz 2006-07-27 17:45 EST ------- I will review this one.
----------------------------------------
Review for release 1.r1564: * RPM name is OK * Source XaraLX-0.7r1564.tar.bz2 is the same as upstream
Needs work: * BuildRequires: perl should not be included (wiki: PackagingGuidelines#Exceptions)
* Build failed in mock autoreconf: failed to run autopoint: No such file or directory
Minor: * Duplicate BuildRequires: automake (by libtool), autoconf (by libtool), perl (by automake), gtk2-devel (by wxGTK-devel)
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
------- Additional Comments From paul@all-the-johnsons.co.uk 2006-07-27 19:31 EST ------- Spec URL: http://www.knox.net.nz/~nodoid/XaraLX.spec
(I've not uploaded a new src.rpm by agreement with the reviewer)
- changed group to Applications/Publishing - fixed autoreconf problem (BR gettext-devel) - removed dupes in BR - removed BR perl - added mime-info - added examples
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
------- Additional Comments From paul@all-the-johnsons.co.uk 2006-07-30 19:48 EST ------- Spec URL: http://www.knox.net.nz/~nodoid/XaraLX.spec SRPM URL: http://www.knox.net.nz/~nodoid/XaraLX-0.7-1.r1614.src.rpm
- bump to newer tarball - Removed the examples from the main package - Created examples subpackage
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
------- Additional Comments From paul@all-the-johnsons.co.uk 2006-07-31 16:14 EST ------- Spec URL: http://www.knox.net.nz/~nodoid/XaraLX.spec Patch URL : Spec URL: http://www.knox.net.nz/~nodoid/xaralx.desktop.patch
- updated spec - fixed desktop file - removed unused (and empty) directory
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
michael@knox.net.nz changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163778 |163779 nThis| |
------- Additional Comments From michael@knox.net.nz 2006-08-01 17:35 EST ------- Looks good Paul. Menu entry shows up now too. APPROVED.
Please remember to close this review request once imported and built.
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: XaraLX - Vector/general purpose graphics utility
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193240
paul@all-the-johnsons.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
------- Additional Comments From paul@all-the-johnsons.co.uk 2006-08-01 18:02 EST ------- Thanks for that and such an easy review process!
package-review@lists.fedoraproject.org
-
Red Hat Bugzilla