[Bug 427481] Review Request: publican - publication tool chain

[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: publican - publication tool chain


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


petersen@xxxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+




------- Additional Comments From petersen@xxxxxxxxxx  2008-02-12 19:52 EST -------
Thanks for the update, Jeff.

For the record here is the rpmlint output now:

 publican.src: W: mixed-use-of-spaces-and-tabs (spaces: line 86, tab: line 1)

which you may want to fix before importing to cvs, and:

 publican.noarch: E: explicit-lib-dependency libxslt
 publican.noarch: W: obsolete-not-provided documentation-devel

which are ok.

(BTW I filed a bug to have po2xml moved from kdesdk to a separate subpackage
to avoid the huge dependency on kdesdk.)

Updating the remaining MUST items from the review in comment 87:

[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.
[+] MUST: The License field in the package spec file must match the actual license.
[+] MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.

[+] MUST: The sources used to build the package must match the upstream source

8e3b3709c1daad154be9759a6c8ee443  publican-0.29.tgz

[+] MUST: Large documentation files should go in a -doc subpackage.
[+] MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section.

So AFAICT all review MUST items are now satisfied, and all the major points
raised have been addressed I think.  I am not a license lawyer obviously,
but the licensing seems reasonable enough to me - if there should
still be issues with the interactions of the licenses I hope they can be
ironed out later if necessary.

Thanks for the review.

Package is APPROVED for inclusion in Fedora.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

_______________________________________________
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]