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/show_bug.cgi?id=447567
Summary: Review Request: msort - Sort files in sophisticated ways Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: terjeros@phys.ntnu.no QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
Spec URL: http://terjeros.fedorapeople.org/msort/msort.spec SRPM URL: http://terjeros.fedorapeople.org/msort/msort-8.45-1.fc9.src.rpm Description:
Msort is a program for sorting files in sophisticated ways. Records need not be single lines. Key fields may be selected by position, tag, or character range. For each key, distinct exclusions, multigraphs, substitutions. and a sort order may be defined. Comparisons may be lexicographic, numeric, by string length, date, or time. Optional keys are supported. Msort uses the Unicode character set and provides full Unicode case-folding. The basic program has a somewhat complex command line interface, but may be driven by an optional GUI.
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: msort - Sort files in sophisticated ways
https://bugzilla.redhat.com/show_bug.cgi?id=447567
terjeros@phys.ntnu.no changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |447566 Summary|Review Request: msort - Sort|Review Request: msort - Sort |files in sophisticated ways |files in sophisticated ways
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: msort - Sort files in sophisticated ways
https://bugzilla.redhat.com/show_bug.cgi?id=447567
------- Additional Comments From terjeros@phys.ntnu.no 2008-05-20 11:20 EST ------- Note: might split the msg (msort graphical) tool to a msort-gui package.
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: msort - Sort files in sophisticated ways
https://bugzilla.redhat.com/show_bug.cgi?id=447567
panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |panemade@gmail.com Status|NEW |ASSIGNED Flag| |fedora-review?
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: msort - Sort files in sophisticated ways
https://bugzilla.redhat.com/show_bug.cgi?id=447567
Bug 447567 depends on bug 447566, which changed state.
Bug 447566 Summary: Review Request: libuninum - Library for converting unicode strings to numbers https://bugzilla.redhat.com/show_bug.cgi?id=447566
What |Old Value |New Value ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
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: msort - Sort files in sophisticated ways
https://bugzilla.redhat.com/show_bug.cgi?id=447567
------- Additional Comments From panemade@gmail.com 2008-05-27 03:49 EST ------- Ok. So msort is GPLv3 and msg is GPLv2.
when I start gui from command line, I see that, # msg msg 1.25 Copyright (C) 2005-2008 William J. Poser. This program is free software; you can redistribute it and/or modify it under the terms of version 2 of the GNU General Public License as published by the Free Software Foundation.
And then when in gui clicked Help->License its showing GPLv3. This is totally incompatible thing. I guess upstream want gui also to be in GPLv3 but missed to remove license text from top of msg.tcl
Can upstream be ping to remove that text from msg.tcl?
look http://fedoraproject.org/wiki/Licensing#head-699ce10b1f5d466cd4c3d61301c3651...
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: msort - Sort files in sophisticated ways
https://bugzilla.redhat.com/show_bug.cgi?id=447567
------- Additional Comments From terjeros@phys.ntnu.no 2008-05-27 04:12 EST -------
Can upstream be ping to remove that text from msg.tcl?
Yes, done.
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: msort - Sort files in sophisticated ways
https://bugzilla.redhat.com/show_bug.cgi?id=447567
------- Additional Comments From terjeros@phys.ntnu.no 2008-05-29 14:31 EST ------- New tarball upstream fixing the issue:
spec: http://terjeros.fedorapeople.org/msort/msort.spec srpm: http://terjeros.fedorapeople.org/msort/msort-8.46-1.fc9.src.rpm
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: msort - Sort files in sophisticated ways
https://bugzilla.redhat.com/show_bug.cgi?id=447567
panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
------- Additional Comments From panemade@gmail.com 2008-05-31 22:54 EST ------- Review: + package builds in mock (rawhide i386). koji build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=638847 + rpmlint is silent for SRPM andfor RPM. + source files match upstream url aa84730eae0691341919db244f821c68 msort-8.46.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + No desktop file installed. + GUI interface is available.
APPROVED.
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: msort - Sort files in sophisticated ways
https://bugzilla.redhat.com/show_bug.cgi?id=447567
terjeros@phys.ntnu.no changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From terjeros@phys.ntnu.no 2008-06-01 06:11 EST -------
APPROVED.
Thanks!
New Package CVS Request ======================= Package Name: msort Short Description: Sort files in sophisticated ways Owners: terjeros Branches: F-8 F-9 InitialCC: Cvsextras Commits: yes
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: msort - Sort files in sophisticated ways
https://bugzilla.redhat.com/show_bug.cgi?id=447567
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2008-06-01 13:30 EST ------- cvs done.
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: msort - Sort files in sophisticated ways
https://bugzilla.redhat.com/show_bug.cgi?id=447567
terjeros@phys.ntnu.no changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
------- Additional Comments From terjeros@phys.ntnu.no 2008-06-04 17:52 EST ------- Built and pushed for F-8, F-9 and rawhide.
package-review@lists.fedoraproject.org