Spec URL: http://fcami.fedorapeople.org/srpms/deafshell.spec SRPM URL: http://fcami.fedorapeople.org/srpms/deafshell-0.1-1.fc13.src.rpm Description: Deaf Shell is intended to be used as a shell for system accounts dedicated to OpenSSH port forwarding, as in: ssh -L [bind_address:]port:host:hostport] user@host ssh -R [bind_address:]port:host:hostport] user@host Deaf Shell will print some information and sleep until receiving a termination signal (Ctrl+C or SIGINT from the user, SIGTERM from the system for instance). It will also print a character every ten seconds.
François, A few comments: 1. The license should be GPLv3+ 2. You could replace the setup and prep sections with just ''%setup -q -n %{name}-%{name}-88674c3' 3. In the build section, it should just be 'make' otherwise rpmlint in unhappy with the unstripped binary 4. In the files section, you should specify the binary as '%{_bindir}deafshell'
Hi Mark, Thanks for the comments. 1. Fixed 2. I don't think so, 88674c3 comes from git and is bound to change at each release tag - what I did will alleviate the problem. 3. I don't think so either - if we strip the binaries during the build, rpmbuild won't generate the debuginfo rpm, which is a bug. Do you have the exact error message? 4. Thanks, fixed. Spec URL: http://fcami.fedorapeople.org/srpms/deafshell.spec SRPM URL: http://fcami.fedorapeople.org/srpms/deafshell-0.1-2.fc13.src.rpm Thank you
François, 3. Whoops, my bad.
Hi François, here are some more comments on your package: - Don't repeat the package/program name in the Summary and keep it concise. Maybe something like the following is more appropriate: "Shell for system accounts dedicated to OpenSSH port forwarding" - Since the source tarball is a snapshot release referring to a changeset of the upstream Git repo, you should adapt the Release field according to http://fedoraproject.org/wiki/PackageNamingGuidelines#Snapshot_packages - Use the %{version} macro in the URL field. - Ensure that Fedora's build flags are used, i.e. add CFLAGS='%{optflags} -lm' to the make statement. - Also, add %{_smp_mflags} to "make" to enable parallel builds.
I've updated the program summary and added CFLAGS. Thank you. However, the source tarball is not a snapshot release, it is related to the 0.1 git tag of the upstream git repository. See https://github.com/deafshell/deafshell/tree/v0.1 - therefore, I don't think I should change the naming to the changeset, though I agree github's tarball generation messes things up. If this is not acceptable to you, I will revert to using the git hash. I normally use %{_smp_mflags} but there is only a single C file to compile. I will update the package if upstream ever becomes more than that. Spec URL: http://fcami.fedorapeople.org/srpms/deafshell.spec SRPM URL: http://fcami.fedorapeople.org/srpms/deafshell-0.1-3.fc14.src.rpm
@François, are you still interested in to work on this package? Your spec and srpm links are dead, and the last commit to upstream's Git was more than two years ago.
Closing this review request now, adding FE-DEADREVIEW.