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=216322
Summary: Review Request: tailor - VCS repository conversion tool Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: dan@danny.cz QAContact: fedora-package-review@redhat.com
Spec URL: http://fedora.danny.cz/tailor.spec SRPM URL: http://fedora.danny.cz/tailor-0.9.26-1.src.rpm
Description: Tailor is a tool to migrate changesets between CVS, Subversion, git, darcs, monotone, Codeville, Mercurial and Baazar-NG repositories.
This script makes it easier to keep the upstream changes merged in a branch of a product, storing needed information such as the upstream URI and revision in special properties on the branched directory.
Notes: - the spec file is based on the ATrpms one - rpmlint is silent on both source and binary rpms
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From Jochen@herr-schmitt.de 2006-11-20 13:24 EST ------- God: + Local build works fine. + Source matches with upstream. * Mock build works fine.
Bad: - Rpmlint shows the following waring on the source package: W: tailor no-%build-section - python-vcpx doesn't contains any documentation - Local test run fails with: Script started on Mo 20 Nov 2006 19:23:47 CET Traceback (most recent call last): File "/usr/bin/tailor", line 24, in ? from vcpx.tests import main ImportError: No module named tests
Script done on Mo 20 Nov 2006 19:23:47 CET
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
Jochen@herr-schmitt.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |Jochen@herr-schmitt.de OtherBugsDependingO|163776 |163778 nThis| |
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From dan@danny.cz 2006-11-20 15:42 EST ------- - (In reply to comment #1)
Bad:
- Rpmlint shows the following waring on the source package:
W: tailor no-%build-section
it used to be possible to omit the build section when there was nothing to do (and my version of rpmlint is silent :-) )
- python-vcpx doesn't contains any documentation
yes, there are really no docs for the subpackage
- Local test run fails with:
Script started on Mo 20 Nov 2006 19:23:47 CET Traceback (most recent call last): File "/usr/bin/tailor", line 24, in ? from vcpx.tests import main ImportError: No module named tests
Script done on Mo 20 Nov 2006 19:23:47 CET
Please, can you tell me, what and how are you trying to run? The included tests during or after the build?
And because there is no COPYING file included, I will ask the upstream to include it in the next versions. The license (GPL) is explicitly mentioned in the README.
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From dan@danny.cz 2007-01-05 08:30 EST ------- Spec URL: http://fedora.danny.cz/tailor.spec SRPM URL: http://fedora.danny.cz/tailor-0.9.27-1.src.rpm
- updated to upstream version 0.9.27
I have requested the inclusion of the COPYING file, but have no reaction yet.
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |medium Priority|normal |medium
------- Additional Comments From dan@danny.cz 2007-04-28 09:09 EST ------- Spec URL: http://fedora.danny.cz/tailor.spec SRPM URL: http://fedora.danny.cz/tailor-0.9.28-1.fc7.src.rpm
- updated to upstream version 0.9.28
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From dan@danny.cz 2007-04-28 10:52 EST ------- Running tests with "tailor test" is not possible on installed package, because the test python module is excluded during the installation run of setup.py. Also local copy of tailor's darcs repository is required.
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- 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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Product|Fedora Extras |Fedora
------- Additional Comments From Jochen@herr-schmitt.de 2007-06-21 16:45 EST ------- Ping Dan
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From dan@danny.cz 2007-06-22 04:30 EST ------- Pong
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From Jochen@herr-schmitt.de 2007-06-24 13:41 EST ------- I have ask to solve the reported issue in Comment #5.
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From dan@danny.cz 2007-06-27 15:09 EST ------- What solution would you like to see? Running tests with installed package is not possible (it is author's intention), so I can offer removing the "test" command from the main script. Do you agree?
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From Jochen@herr-schmitt.de 2007-06-27 15:18 EST ------- OK, If is impossible to run the tests, then remove it.
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From dan@danny.cz 2007-06-27 15:48 EST ------- Spec URL: http://fedora.danny.cz/tailor.spec SRPM URL: http://fedora.danny.cz/tailor-0.9.28-2.fc7.src.rpm
- disabled running tests on installed 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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From Jochen@herr-schmitt.de 2007-06-28 11:49 EST ------- Good: + Package meets naming guidlines + Spec file match with name of the package + RPM macros are used consistently + Package contains proper buildroot + Sub packages are defined properly + Package is llcensed on the terms of the GPL. + Header of source files contains references to the GPL + Tar ball metches with upstream (md5sum: 82033c16e895d51ae5893def06c59d29) + Package contains proper %defattr and file permissions + Filelist contains no duplicated entry + Package contains no files or directory own by other packages + %doc section is small + %doc section doen't affect runtime + Package contains proper %clean section + Buildroot will clean on the beginning of the %install section + Local build works fine. + Changelog section looks ok + Rpmlint is quite on source package + Mock build works fine for Devel and F-7 (x86_64)
Bad: - Package doesn't contains verbatim copy of the license text (Please ask upstream to include a copy of the GPL license text) - Rpmlint complaints python-vcpx package: E: python-vcpx non-executable-script /usr/lib/python2.5/site-packages/vcpx/repository/p4/p4lib.py 0644 This text file contains a shebang or is located in a path dedicated for executables, but lacks the executable bits and cannot thus be executed. If the file is meant to be an executable script, add the executable bits, otherwise remove the shebang or move the file elsewhere.
You should remove the shebang line from the file complaint by Rpmlint.
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From dan@danny.cz 2007-06-28 13:08 EST ------- Spec URL: http://fedora.danny.cz/tailor.spec SRPM URL: http://fedora.danny.cz/tailor-0.9.28-3.fc7.src.rpm
- removed the shebang line from non-executable script
I have also asked the upstream author to include the copy of GPL into the distributed sources.
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From jochen@herr-schmitt.de 2007-07-01 15:03 EST ------- Hello,
Now the Rpmlint output is OK for me, so I can APPROVED the package.
Unfortanately, I can't change the fedora-review flag thue a technical issue.
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
------- Additional Comments From wolfy@nobugconsulting.ro 2007-07-01 18:17 EST ------- Setting the fedora+ flag on behalf of Jochen
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
wolfy@nobugconsulting.ro changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
dan@danny.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From dan@danny.cz 2007-07-03 03:26 EST ------- New Package CVS Request ======================= Package Name: tailor Short Description: VCS repository conversion tool Owners: dan@danny.cz Branches: F-7 devel InitialCC: dan@danny.cz
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
dan@danny.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163778 | nThis| |
------- Additional Comments From dan@danny.cz 2007-07-03 03:28 EST ------- Thanks for the review. I am removing the FE-REVIEW blocker bug now.
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
wtogami@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
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: tailor - VCS repository conversion tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216322
dan@danny.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
------- Additional Comments From dan@danny.cz 2007-07-04 07:31 EST ------- package imported and built
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=216322
Dan Horák dan@danny.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
--- Comment #19 from Dan Horák dan@danny.cz 2008-11-25 07:43:44 EDT --- Package Change Request ====================== Package Name: tailor New Branches: EL-5 Owners: sharkcz
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=216322
Dennis Gilmore dennis@ausil.us changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #20 from Dennis Gilmore dennis@ausil.us 2008-11-25 11:53:30 EDT --- CVS Done
package-review@lists.fedoraproject.org