SPEC URL: http://xiashing.fedorapeople.org/SPECS/sympa.spec SRPM URL: http://xiashing.fedorapeople.org/SRPMS/sympa-6.0b.2-1.fc11.src.rpm Description: Sympa is an electronic mailing list manager. It is used to automate list management functions such as subscription, moderation, archive and shared document management. It also includes management functions which would normally require a substantial amount of work.
%file section must use macros. /sample is a big no-no. The apache conf is missing list creation from the UI is probably not working etc... This needs a lot more work. I did not really take a look at sympa 6.0 yet, but I worked on packaging 5.4 some time ago and it needed much more magic, even if 6.0 is supposed to be much more friendly wrt FHS now. My work is far from finished, hence I did not submit it, but you might want to take a look. http://www.bachelot.org/fedora/SPECS/sympa.spec http://www.bachelot.org/fedora/SRPMS/sympa-5.4.7-3.fc10.src.rpm I'll try to help as much as I can with this review.
A couple of quick comments: * the license tag must be GPLv2+ * This is a pre-release, so you have to take care of the version and release number. See: http://fedoraproject.org/wiki/Packaging/NamingGuidelines#NonNumericRelease - Version: 6.0 - Release: 0.X.b2%{?dist} (X is the revision number to increase) * remove the build target (.f11) from the changelog entry
Xavier, What should I put instead of /sample? RPM probably won't recognize that as any macro. When I built the source, it didn't have an apache.conf or httpd.conf file. Martin, I removed .f11 from the changelog and modified the license tag. When using the version and release you suggested, RPM doesn't want to build the file as it can't cd into the directory. Also, I don't think it really matters since Sympa is a hard package, but I got a scratch build to build properly: https://koji.fedoraproject.org/koji/taskinfo?taskID=1690769
(In reply to comment #3) > When using the > version and release you suggested, RPM doesn't want to build the file as it > can't cd into the directory. Right. In this case you have to tell %setup the actual directory name: %setup -q -n %{name}-%{version}b.2
New SPEC: http://xiashing.fedorapeople.org/SPECS/sympa.spec New SRPM: http://xiashing.fedorapeople.org/SRPMS/sympa-6.0-0.3.b2.fc11.src.rpm %doc was updated dist-rawhide fails with BuilldrootError: https://koji.fedoraproject.org/koji/taskinfo?taskID=1690991 dist-f11 completes successfully: https://koji.fedoraproject.org/koji/taskinfo?taskID=1690986 So a few more things to be done here: /sample in %file has to be changed. The apache conf is missing? I don't think it needs an apache conf
You need to pass more option to %configure to have the files installed in the proper location. Take a look at the sympa install doc. (--enable-fhs, --with-defaultdir, etc...) You must use macros in the %files section. https://fedoraproject.org/wiki/Packaging:RPMMacros The NVR (name-version-release) in changelog entries must match the actual NVR defined with Name: Version: Release tags. The apache conf, log rotate conf, etc... are not provided in the tarball, but you need to add them to get a fully functional software. The provided initscripts are also not matching the fedora guidelines (default to enabled, no reload target iirc) I ended replacing them. You need to create a sympa user to run all the scripts. You need to create an additional mail alias file, etc... You really should take a look at the specfile and srpm I'm linking in my first comment. It is based on an older version of sympa and still far from perfect, but it will help you get a better idea of what is needed to get both a functional software and a specfile/srpm matching the Fedora guidelines.
May you move to 6.0.1 and address issues mentioned in comment #6. As soon as you do that I will do a detailed review. Thanks,
It has been long since any response. I will move close it as deferred if no response in a week. Regards, Rakesh Pandit
Deferring it for time being. In case you find some time for it later on, pleas reopen the bug and update. Regards, Rakesh Pandit