Fedora Merge Review: festival http://cvs.fedora.redhat.com/viewcvs/devel/festival/ Initial Owner: davidz
- rpmlint checks return: festival.spec:212: W: unversioned-explicit-provides festival-voice festival.spec:213: W: unversioned-explicit-provides festvox-kallpc16k festival.spec:221: W: unversioned-explicit-provides festival-voice festival.spec:222: W: unversioned-explicit-provides festvox-kedlpc16k festival.spec:230: W: unversioned-explicit-provides festival-voice festival.spec:238: W: unversioned-explicit-provides festival-voice festival.spec:246: W: unversioned-explicit-provides festival-voice festival.spec:254: W: unversioned-explicit-provides festival-voice festival.spec:262: W: unversioned-explicit-provides festival-voice festival.spec:270: W: unversioned-explicit-provides festival-voice festival.spec:278: W: unversioned-explicit-provides festival-voice festival.spec:286: W: unversioned-explicit-provides festival-voice The specfile contains an unversioned Provides: token, which will match all older, equal, and newer versions of the provided thing. This may cause update problems and will make versioned dependencies, obsoletions and conflicts on the provided thing useless -- make the Provides versioned if possible. Should be fixed. festival.spec:1080: W: macro-in-%changelog %{festivalversion} festival.spec:1157: W: macro-in-%changelog %{_bindir} Macros are expanded in %changelog too, which can in unfortunate cases lead to the package not building at all, or other subtle unexpected conditions that affect the build. Even when that doesn't happen, the expansion results in possibly "rewriting history" on subsequent package revisions and generally odd entries eg. in source rpms, which is rarely wanted. Avoid use of macros in %changelog altogether, or use two '%'s to escape them, like '%%foo'. Trivial to fix. festival.spec:746: W: mixed-use-of-spaces-and-tabs (spaces: line 18, tab: line 746) The specfile mixes use of spaces and tabs for indentation, which is a cosmetic annoyance. Use either spaces or tabs for indentation, not both. Trivial to fix. Lots of no-manpage, wrong end of line encoding, no-shebang or spurious executable perms. festival-devel.x86_64: W: no-dependency-on festival/festival-libs/libfestival festival-lib.x86_64: E: explicit-lib-dependency festival-speechtools-libs You must let rpm find the library dependencies by itself. Do not put unneeded explicit Requires: tags. Fix. festival-speechtools-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libestbase.so.1.2.96.1 exit.5 This library package calls exit() or _exit(), probably in a non-fork() context. Doing so from a library is strongly discouraged - when a library function calls exit(), it prevents the calling program from handling the error, reporting it to the user, closing files properly, and cleaning up any state that the program has. It is preferred for the library to return an actual error code and let the calling program decide how to handle the situation. festival-speechtools-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libestools.so.1.2.96.1 exit.5 This library package calls exit() or _exit(), probably in a non-fork() context. Doing so from a library is strongly discouraged - when a library function calls exit(), it prevents the calling program from handling the error, reporting it to the user, closing files properly, and cleaning up any state that the program has. It is preferred for the library to return an actual error code and let the calling program decide how to handle the situation. festival-speechtools-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libeststring.so.1.2 exit.5 This library package calls exit() or _exit(), probably in a non-fork() context. Doing so from a library is strongly discouraged - when a library function calls exit(), it prevents the calling program from handling the error, reporting it to the user, closing files properly, and cleaning up any state that the program has. It is preferred for the library to return an actual error code and let the calling program decide how to handle the situation. Should be fixed if at all possible. - package meets naming guidelines - package meets packaging guidelines - license ( MIT and GPL+ and TCL ) OK, text in %doc, matches source - spec file legible, in am. english - source matches upstream Though it's ancient. . . and the doc versions still don't match. - package compiles on devel (x86_64) - no missing BR - no unnecessary BR - no locales - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - no need for .desktop file - devel package ok - no .la files - post/postun ldconfig ok - devel requires base package n-v-r Other than the above, not much to do, let me know if you want me to commit anything.
Ping?
Fixed most things except the exit() calls. Currently orphaned, Mattias, do you intend to take this? If so, can you take a look?
Mass reassigning all merge reviews to their component. For more details, see this FESCO ticket: https://fedorahosted.org/fesco/ticket/1269 If you don't know what merge reviews are about, please see: https://fedoraproject.org/wiki/Merge_Reviews How to handle this bug is left to the discretion of the package maintainer.
This bug appears to have been reported against 'rawhide' during the Fedora 23 development cycle. Changing version to '23'. (As we did not run this process for some time, it could affect also pre-Fedora 23 development cycle bugs. We are very sorry. It will help us with cleanup during Fedora 23 End Of Life. Thank you.) More information and reason for this action is here: https://fedoraproject.org/wiki/BugZappers/HouseKeeping/Fedora23
The Festival package in Fedora needs a lot of work to update to a newer release, fix a lot of bugs, and generally clean up the packaging. And, despite best intentions, we just haven't had the developer interest in doing so. The current plan is to retire Festival from Fedora. See thread at https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/message/UOL4ETKKOFGTFZZ36V726OF7UTHYMEYP/ for discussion. Despite being large, slow, and fragile, Festival is interesting software and as far as I know can produce the best quality results of any open-source TTS system. It would be nice if there is interest in continuing it, but that should probably be done from a clean slate in any case. In the meantime, I think it's most honest to close the currently-open bugs as "WONTFIX". Thanks everyone for your reports and effort in making Fedora better.