Overview
Request 688046 superseded
Thanks for the review Stefan. I adjusted everything according to your feedback :)
I just let the COPYING file in the geda-gaf as I think it fits there better together with the README and the NEWS file.
If you still want to have it in geda-base, feel free to accept the SR and change it afterwards :)
- Created by jbrielmaier
- In state superseded
- Supersedes 687650
- Superseded by 688500
Thanks for working on this, some remarks:
Changelog:
- Sentences should start with a capital letter
- Check indentation for continued lines
Strange entry "backends.", third line
%defattr is obsolete, just remove it
COPYING should be part of base, and use the %license macro
Request History
jbrielmaier created request
Thanks for the review Stefan. I adjusted everything according to your feedback :)
I just let the COPYING file in the geda-gaf as I think it fits there better together with the README and the NEWS file.
If you still want to have it in geda-base, feel free to accept the SR and change it afterwards :)
StefanBruens declined request
Fails to build, you still install the COPYING manually into the %_docdir
The reason for putting the COPYING into the base package is to have it installed as soon as any other part is installed.