[Bug 840878] Review Request: liberation-narrow-fonts - Sans-serif Narrow fonts to replace commonly used Microsoft Arial Narrow
https://bugzilla.redhat.com/show_bug.cgi?id=840878 Parag AN(पराग) changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #3 from Parag AN(पराग) --- Review:- + Koji f18 scratch build -> http://koji.fedoraproject.org/koji/taskinfo?taskID=4325263 + rpmlint on rpms gave 2 packages and 0 specfiles checked; 0 errors, 0 warnings. + source verified with upstream as (sha1sum) 4db1abfdb6e41bdc9a461fc7dfecf996ff1d66e6 liberation-fonts-1.07.2.tar.gz 4db1abfdb6e41bdc9a461fc7dfecf996ff1d66e6 ../SOURCES/liberation-fonts-1.07.2.tar.gz + License is same as old package license and valid. APPROVED. You need to import this package and new update of liberation-fonts in same rawhide update. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 840878] Review Request: liberation-narrow-fonts - Sans-serif Narrow fonts to replace commonly used Microsoft Arial Narrow
https://bugzilla.redhat.com/show_bug.cgi?id=840878 --- Comment #2 from Pravin Satpute --- hmm, lots of changes happened after liberation-fonts last review. I will do suggested changes in liberation-fonts packages as wellupdated spec and srpm. Wondering still liberation fonts does not have .conf file, task pending on me though SRPM Url : http://pravins.fedorapeople.org/liberation-narrow-fonts-1.07.2-8.fc17.src.rpm SPEC Url : http://pravins.fedorapeople.org/liberation-narrow-fonts.spec -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 840878] Review Request: liberation-narrow-fonts - Sans-serif Narrow fonts to replace commonly used Microsoft Arial Narrow
https://bugzilla.redhat.com/show_bug.cgi?id=840878
--- Comment #1 from Parag AN(पराग) ---
suggestions:
1) Use %global instead of %define.
2) cp commands should preserve timestamp
3) use actual commands for {_ln_s}
--
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 840878] Review Request: liberation-narrow-fonts - Sans-serif Narrow fonts to replace commonly used Microsoft Arial Narrow
https://bugzilla.redhat.com/show_bug.cgi?id=840878 Parag AN(पराग) changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com Assignee|nob...@fedoraproject.org|panem...@gmail.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
