Bug 959509 - Review Request: trac10 - Enhanced wiki and issue tracking system
Summary: Review Request: trac10 - Enhanced wiki and issue tracking system
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Dan Mashal
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-05-03 16:24 UTC by Gwyn Ciesla
Modified: 2013-06-21 19:39 UTC (History)
5 users (show)

Fixed In Version: trac10-1.0.1-4.el6
Clone Of:
Environment:
Last Closed: 2013-06-21 19:39:24 UTC
Type: Bug
Embargoed:
dan.mashal: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Gwyn Ciesla 2013-05-03 16:24:30 UTC
This will be for EPEL 6 exclusively.

Description:
Trac is an integrated system for managing software projects, an
enhanced wiki, a flexible web-based issue tracker, and an interface to
the Subversion revision control system.  At the core of Trac lies an
integrated wiki and issue/bug database. Using wiki markup, all objects
managed by Trac can directly link to other issues/bug reports, code
changesets, documentation and files.  Around the core lies other
modules, providing additional features and tools to make software
development more streamlined and effective.

SRPM: http://fedorapeople.org/~limb/review/trac/trac10-1.0.1-3.el6.src.rpm
SPEC: http://fedorapeople.org/~limb/review/trac/trac10.spec

Comment 1 Volker Fröhlich 2013-05-04 09:32:12 UTC
You can drop defattr, clean section, rm in the install section and the buildroot definition.

python_sitelib should be defined already and I think you should BR python2-devel: https://fedoraproject.org/wiki/Packaging:Python

Please use the sysconfdir macro instead of /etc.

Comment 2 Gwyn Ciesla 2013-05-07 14:19:16 UTC
This is intended for EL-6 and possibly EL-5, so I'll take those under advisement, but I will macroise /etc.  Thanks!

Comment 3 Dan Mashal 2013-05-07 14:43:54 UTC
Thanks for replying to that.

Naming: OK
Licensing: OK
SRPM builds just fine on KOJI

Can you provide a quick easy way to test this?

Thanks,
Dan

Comment 4 Gwyn Ciesla 2013-05-07 14:48:55 UTC
You mean besides using it to set up a Trac instance on a EL-6 machine?

Comment 5 Dan Mashal 2013-05-07 14:51:52 UTC
(In reply to comment #4)
> You mean besides using it to set up a Trac instance on a EL-6 machine?

No. Is there a quick setup .doc (I haven't looked too hard), or maybe just a link to a site? 

That way I can say I tested it and it works. I have a CentOS 6.4 32bit machine ready to go.

I'm not too familiar with trac. Never set it up myself. I just use it when creating tickets.

Comment 6 Gwyn Ciesla 2013-05-07 14:57:54 UTC
I'd start here.

http://trac.edgewall.org/wiki/TracInstall#CreatingaProjectEnvironment

Comment 7 Seth Kress 2013-05-28 13:39:05 UTC
If it helps, I have used the RPM posted (changes requested not included) and have been testing it for a few weeks.

I have setup 2 new environments and migrated one trac .12 envronment up to 1.0.  No anomolies experianced.

Hope this helps.

Comment 8 Gwyn Ciesla 2013-05-28 14:18:29 UTC
Excellent, thanks!

Comment 9 Dan Mashal 2013-06-04 02:24:50 UTC
(In reply to Seth Kress from comment #7)
> If it helps, I have used the RPM posted (changes requested not included) and
> have been testing it for a few weeks.
> 
> I have setup 2 new environments and migrated one trac .12 envronment up to
> 1.0.  No anomolies experianced.
> 
> Hope this helps.

Thanks. Approved!

With that being said you can process your own SCM request. :)

Comment 10 Dan Mashal 2013-06-04 02:25:20 UTC
Also, sorry for the delay.

Comment 11 Gwyn Ciesla 2013-06-04 12:14:04 UTC
Thanks, much appreciated.  But before I do that, can I have you go into a bit more detail about what you reviewed?  It's a bit sparse.

Comment 12 Dan Mashal 2013-06-04 14:17:39 UTC
I checked licensing, RPMlint, koji builds, all check out ok, I just wanted to test it and work has been a nightmare. I can paste a fedora-review later today, but you have a proven track record. I did go through the spec, rpm, licensing and rpmlint.

Comment 13 Gwyn Ciesla 2013-06-04 18:49:32 UTC
Please, do remember, we're evaluating the package at hand, not whatever I may have done in the past.  Absolutely anyone can make mistakes, regardless of prior work.  That's the whole point. :)  Thank you for your confidence.  I appreciate it, but will await the fedora-review output.

Trust, but verify, as they say. ;)

Comment 14 Dan Mashal 2013-06-05 08:39:36 UTC
[dan@localhost srpm]$ rpmlint trac10-1.0.1-3.el6.src.rpm 
trac10.src: W: spelling-error %description -l en_US Trac -> Traci, Tracy, Tran
trac10.src: W: spelling-error %description -l en_US changesets -> change sets, change-sets, changeless
trac10.src:36: W: macro-in-comment %patch0
1 packages and 0 specfiles checked; 0 errors, 3 warnings.
[dan@localhost srpm]$ rpmlint trac10.spec 
trac10.spec:36: W: macro-in-comment %patch0
0 packages and 1 specfiles checked; 0 errors, 1 warnings.


Scratch build here:


 koji build --scratch dist-6E-epel trac10-1.0.1-3.el6.src.rpm 
Uploading srpm: trac10-1.0.1-3.el6.src.rpm
[====================================] 100% 00:00:08   3.34 MiB 399.02 KiB/sec
Created task: 5469395
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=5469395
Watching tasks (this may be safely interrupted)...
5469395 build (dist-6E-epel, trac10-1.0.1-3.el6.src.rpm): open (buildvm-20.phx2.fedoraproject.org)
  5469398 buildArch (trac10-1.0.1-3.el6.src.rpm, noarch): free
  5469398 buildArch (trac10-1.0.1-3.el6.src.rpm, noarch): free -> open (buildvm-22.phx2.fedoraproject.org)
  5469398 buildArch (trac10-1.0.1-3.el6.src.rpm, noarch): open (buildvm-22.phx2.fedoraproject.org) -> closed
  0 free  1 open  1 done  0 failed
5469395 build (dist-6E-epel, trac10-1.0.1-3.el6.src.rpm): open (buildvm-20.phx2.fedoraproject.org) -> closed
  0 free  0 open  2 done  0 failed

5469395 build (dist-6E-epel, trac10-1.0.1-3.el6.src.rpm) completed successfully


I think the BSD license coversr the MIT if you want to be super cautious about it, change the license to BSD + MIT (or X11). Other than that we have a comment from someoen confirming that the package works. 

NOTE:

Okay so one thing, does the BSD license cover MIT/X11? That's all I have minus the fact that Requires:       python-genshi06 >= 0.6 is obsoleted in Fedora.

NOTE:

This is not for Fedora it's for EPEL.

SHOULD: 

Get rid of unused patch files in the spec.

Other than that looks GOOD to ME. And confirmed working by another comment.

Comment 15 Dan Mashal 2013-06-05 08:40:57 UTC
Also note, fedora-review fails, (packages should build on rawhide).

Maybe add a provides for Fedora builds?

Comment 16 Gwyn Ciesla 2013-06-05 13:20:29 UTC
It says modified BSD in README, and they're all in the same family, so I'll leave it.  Thanks!

New Package SCM Request
=======================
Package Name: trac10
Short Description: Enhanced wiki and issue tracking system
Owners: limb
Branches: el6
InitialCC:

Comment 17 Gwyn Ciesla 2013-06-05 13:33:53 UTC
Git done (by process-git-requests).

Comment 18 Gwyn Ciesla 2013-06-05 14:19:59 UTC
Sigh.  And now it won't build, test failure.  Will fix prior to import.

Comment 19 Dan Mashal 2013-06-05 14:29:55 UTC
Odd...there was a few minutes ago. Don't know if that's related but the scratch build is less than 12 hours old

Comment 20 Dan Mashal 2013-06-05 14:30:20 UTC
Sorry, there was an infrastructure issue*

Comment 21 Gwyn Ciesla 2013-06-05 14:37:16 UTC
I meant locally, not koji.

Comment 22 Dan Mashal 2013-06-05 14:40:35 UTC
Odd. Let me know if I can help in anyway. I'll try and be available from the office.

Comment 23 Gwyn Ciesla 2013-06-05 14:43:38 UTC
Really odd, worked in mock.  Must be something else I have installed.  I'll import and build, should be OK.

Comment 24 Dan Mashal 2013-06-05 14:44:45 UTC
Happy limbs are good. :)

Comment 25 Fedora Update System 2013-06-05 14:58:29 UTC
trac10-1.0.1-4.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/trac10-1.0.1-4.el6

Comment 26 Fedora Update System 2013-06-06 16:57:30 UTC
trac10-1.0.1-4.el6 has been pushed to the Fedora EPEL 6 testing repository.

Comment 27 Fedora Update System 2013-06-21 19:39:24 UTC
trac10-1.0.1-4.el6 has been pushed to the Fedora EPEL 6 stable repository.


Note You need to log in before you can comment on or make changes to this bug.