Peter Lemenkov wrote: > Hello All! > Just FYI - I finally decided to push SEMS to official Fedora/EPEL > repositories - any comments or suggestions are welcome: > > https://bugzilla.redhat.com/show_bug.cgi?id=45895 > > http://peter.fedorapeople.org/sems.spec > http://peter.fedorapeople.org/sems-1.0.0-0.1.svn.fc9.src.rpm > > It's based on current svn branch 1.0.0. > > After all I can say that SEMS buildsystem is one of the hardest to > understand I ever seen. I spent many hours trying to force SEMS to > install according to our infrastructure rules. :) Still one thing > unresolved - wav-files, corresponding to respective IVR-mdules still > doesn't install. > >
You may be right.... That is one thing on which we haven't spent much cycles for quite a long time. We may need some build system specialist having a look at this mess. > Another one note is a mess with the source layout - what's the > difference between apps and code/plug-in content? Seems that it would > be better to merge them. On the other hand, content of apps/examples > directory should be moved to its own director, probably at top level > (sems-1.0.0/examples). Before such move, content of > apps/examples/tutorial should be moved one directory up and merget > therefore with examples content. It's just my opinion, though. > > code/plug-in is supposed to contain general purpose plug-in that will always be loaded, or should be part of the base version. The apps directory contains all the plug-in which can be used separatly for specific purposes. Does it make sense? > There is also a discrepancy in placing of Readme.* files - some of > them placed in docs, while others - in the directories of their > respective modules I personally think that the latter variant is > better (all information about particular module placed in one place). > That's also just my personal opinion. > > And one simple note - you shipped init-script for RedHat, which quite > outdated. According to latest RedHat/Fedora policy only services for a > vital system should have a default runlevel list in their initscript. > Easy to fix, actually (see patch): > > http://peter.fedorapeople.org/sems--initscript_fix.diff > > Many thanks for your work and comments! -Raphael. _______________________________________________ Semsdev mailing list [email protected] http://lists.iptel.org/mailman/listinfo/semsdev
