Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add lightscreen.spec to enable builds for (S)RPM #41

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Darksider3
Copy link

@Darksider3 Darksider3 commented Apr 23, 2018

Hey!
I've created for a friend that *.spec file.
It just enables lightscreen to be build trough RPKG/RPM and build corresponding packages. As well, it would be awesome when you could merge the success-fix: #40, because it wont build anywhere without it.
There are already some done on
COPR: https://copr.fedorainfracloud.org/coprs/darksider3/lightscreen/package/lightscreen/
Open Build System(SUSE): https://build.opensuse.org/package/show/home:darksider3/Lightscreen (this one actually only works for factory-versions yet because there is no qt5.9 in the other ones)
Thanks.

See changelog: 

* Wed Apr 25 2018 darksider3 <[email protected]> - 1.6.2
- Change all Uppercase Lightscreen to name-variable
- use uncompress-macro, not unzip
- use S/P-macro instead of SHAs
- use global instead of define
- use __cp-macro instead of cp
- use __install-macro instead of install
- __make-macro instead of make
- __mkdir_p-macro instead of mkdir-p
- __patch-macro instead of patch
- delete macro in comments
- include README.md and LICENSE!
- remove executable flag from doc-files.
- reindent changelog
@Darksider3
Copy link
Author

Darksider3 commented Apr 30, 2018

I've also added 2 patches. One is that one described in request Nr. 40, the other one introduces some default return, because open suse's open build system doesn't like functions without guaranteed returns

not needed anymore(updatet to (uppercase) Lightscreen.spec)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant