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

Ci integration #10

Merged
merged 3 commits into from Oct 2, 2013
Merged

Ci integration #10

merged 3 commits into from Oct 2, 2013

Conversation

jreidinger
Copy link
Member

basic setup to allow CI integration and add basic rake tasks

lslezak and others added 3 commits September 19, 2013 19:25
use *.spec file with RPM macros instead

- 3.1.0
Conflicts:
	package/yast2-storage.changes
	yast2-storage.spec.in
@aschnell
Copy link
Member

aschnell commented Oct 2, 2013

Was the %clean section removed on purpose?
Wouldn't it be a good idea to introduce a %check section?

@jreidinger
Copy link
Member Author

@lslezak feel free to also add comments

@aschnell clean section is removed on purpose as rpm do it now by default ( removing build root ). Introducing %check section would be great, but it doesn't work with old testsuite as it need installed package. so if we use everywhere new tests that can run without install, then we can introduce it.

aschnell added a commit that referenced this pull request Oct 2, 2013
@aschnell aschnell merged commit 3d3da66 into master Oct 2, 2013
@shundhammer shundhammer deleted the ci_integration branch March 23, 2017 13:07
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.

None yet

3 participants