[Bug 1548744] Review Request: unicode-cldr - Unicode Common Locale Data Repository

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



--- Comment #3 from Neal Gompa <ngompa13@xxxxxxxxx> ---
Based on the report, there are a couple of things to review:

* You include the ASL license file and the ICU license, but they're not
declared in the tag. Moreover, based on the readme and other supporting
information[1], it appears that the only license file you need to include for
the two subpackages you have is unicode-license.txt. Please revise accordingly.

[1]:
http://metadata.ftp-master.debian.org/changelogs/main/u/unicode-cldr-core/unicode-cldr-core_32.0.1-1_copyright

* The unowned directories above these two need to be properly "owned". This is
done with the %dir directive[2]. This will be co-owned by both subpackages,
since they are independent ones.

[2]: http://ftp.rpm.org/max-rpm/s1-rpm-inside-files-list-directives.html

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux