[Bug 1069257] Review Request: fparser - Function parser library for C++

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

 



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



--- Comment #29 from Till Hofmann <hofmann@xxxxxxxxxxxxxxxxxxx> ---
Thanks for the patch!

I've included the patch and updated the SPEC file, could you check it before I
update the package?
SRPM: http://thofmann.fedorapeople.org/fparser-4.5.1-6.fc20.src.rpm
SPEC: http://thofmann.fedorapeople.org/fparser.spec
patch: http://thofmann.fedorapeople.org/fparser.autotools.patch

I wasn't sure whether to include the autotools files or only the generated
files. In https://fedoraproject.org/wiki/PackagingDrafts/AutoConf, it is
recommended to only include the generated files, but in the corresponding
discussion, many people disagreed. Since we maintain the autotools files
ourselves, we shouldn't get into trouble with incompatible autotools versions.
Following the suggestions in the discussion, I included the autoconf files and
changed the SPEC file appropiately.

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