Bug 1358956
| Summary: | Review Request: sstp-client - Secure Socket Tunneling Protocol(SSTP) Client | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Marcin Zajaczkowski <mszpak> |
| Component: | Package Review | Assignee: | Lubomir Rintel <lkundrak> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | lkundrak, package-review |
| Target Milestone: | --- | Flags: | lkundrak:
fedora-review+
|
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2016-09-01 19:08:00 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Marcin Zajaczkowski
2016-07-21 22:23:26 UTC
I applied a workaround for the issue with Rpath, but there are still some fedora-review/rpmlint issues:
Rpmlint
-------
Checking: sstp-client-1.0.10-5.fc24.x86_64.rpm
sstp-client-devel-1.0.10-5.fc24.x86_64.rpm
sstp-client-debuginfo-1.0.10-5.fc24.x86_64.rpm
sstp-client-1.0.10-5.fc24.src.rpm
sstp-client.x86_64: W: spelling-error %description -l en_US pptp -> PT
sstp-client.x86_64: W: spelling-error %description -l en_US pon -> pin, pom, on
sstp-client.x86_64: W: spelling-error %description -l en_US poff -> off, toff, poof
sstp-client.x86_64: E: missing-call-to-setgroups-before-setuid /usr/sbin/sstpc
sstp-client.x86_64: E: missing-call-to-chdir-with-chroot /usr/sbin/sstpc
sstp-client.x86_64: W: dangerous-command-in-%postun rm
sstp-client-devel.x86_64: W: only-non-binary-in-usr-lib
sstp-client.src: W: spelling-error %description -l en_US pptp -> PT
sstp-client.src: W: spelling-error %description -l en_US pon -> pin, pom, on
sstp-client.src: W: spelling-error %description -l en_US poff -> off, toff, poof
4 packages and 0 specfiles checked; 2 errors, 8 warnings.
> E: missing-call-to-setgroups-before-setuid /usr/sbin/sstpc
It should be easy to fix, but I haven't been programming in C for years and I reported it upstream:
https://sourceforge.net/p/sstp-client/discussion/1499217/thread/d1508589/
> sstp-client.x86_64: E: missing-call-to-chdir-with-chroot /usr/sbin/sstpc
It's strange as there is a chdir() call:
https://sourceforge.net/p/sstp-client/code/200/tree/trunk/src/sstp-util.c#l364
> W: only-non-binary-in-usr-lib
There is just a symlink to .so file from the main package
> W: dangerous-command-in-%postun rm
It seems to be required for clean up after %pre operations - maybe it can be simplified somehow?
Hmm, strange thing: Source checksums ---------------- http://downloads.sourceforge.net/project/sstp-client/sstp-client/1.0.10/sstp-client-1.0.10.tar.gz : CHECKSUM(SHA256) this package : 5f9084d8544c42c806724a4e70d039d8cb7b0ea06be8ea9cc5120684d4e0d424 CHECKSUM(SHA256) upstream package : 0fa26e565c169723e288b52de47a0cb6d838450b26d404726330adde3147eb4a diff -r also reports differences I will investigate it tomorrow. In the meantime, feel free to make a general review of the other parts :). I don't why mock build returns that checksum error. The package downloaded with wget from mentioned URL has the same checksum as the tarball in the official Fedora RPM. Maybe it's only an issue on my machine... The package indeed looks correct to me. It was in Fedora for some time and only removed recently, already meets our quality standard. For me the pristine tarball is indeed 5f290355187e5ce0423fb7e388e65b9d; not sure what went wrong there. But it corresponds to what's in SVN as well. APPROVED sstp-client-1.0.10-5.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-3e73d66fc7 sstp-client-1.0.10-5.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2016-59303cb075 sstp-client-1.0.10-5.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-3e73d66fc7 sstp-client-1.0.10-5.fc25 has been pushed to the Fedora 25 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-59303cb075 sstp-client-1.0.10-5.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report. sstp-client-1.0.10-5.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report. |