[Bug 1279175] Review Request: innoextract - Tool to extract installers created by Inno Setup

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

 



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



--- Comment #2 from Alexandre Detiste <alexandre.detiste@xxxxxxxxx> ---
1) ok

2) ok

I also keep an history of my changes on GitHub, is there a more appropriate
location where to put this specfile,
or at least shouldn't this location be linked back from the specfile
in some machine-readable way ?

  https://github.com/a-detiste/fedora/commits/master

3) rpmlint:

- I've fixed the tab vs spaces.


- Didn't found a way to silence the spurious warning about "Inno" being a typo.
  "W: spelling-error %description -l en_US Inno -> In no, In-no, Inn"

 
https://ask.fedoraproject.org/en/question/10738/rpmlint-override-in-spec-file/


- It's the tarball at GitHub, the one with the
  "Please do not use the automatically generated source archives below"
  disclaimer, mea culpa.


Thanks for the review

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]