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

#GPII-1740: Implement automatic build of Linux Package #78

Closed
wants to merge 7 commits into from

Conversation

amatas
Copy link

@amatas amatas commented Apr 28, 2016

This PR allows to build a bundle RPM package of GPII-Linux and GPII-universal from a snapshot of GIT. The package installs all the files at /opt/gpii-linux of the distribution and sets the launch of GPII when the user logs in.

More info at Jira issue: #GPII-1740


To build a RPM package run the following command at the root of the git repository:

docker run --rm -it \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any way this command could be automated (or at least simplified) using a grunt task? So that a developer could just run "grunt buildrpm" and have everything run automatically?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@colinbdclark done! :
grunt buildrpm builds a package in a Fedora environment
grunt buildRpmDocker builds a package using a Fedora Docker container

We use Docker to build the RPM package in a clean environment, this also avoids
to polute your environment.

To build a RPM package run the following command at the root of the git repository:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How would this work for someone using OS X or Windows? Should they use a Linux VM for this task? Those details should be explained here.

@colinbdclark
Copy link
Member

It looks like this pull request probably needs to be updated to the latest master, is that correct, @amatas?

@amatas
Copy link
Author

amatas commented May 30, 2016

Indeed @colinbdclark , it needs some changes that @avtar mentioned. I will incorporate them ASAP.

@colinbdclark
Copy link
Member

Any updates on this one, @amatas? I know you have a lot on your plate.

@amatas
Copy link
Author

amatas commented Jun 27, 2016

#84 Created with the latest changes of the repository merged. I'll close this PR.

@amatas amatas closed this Jun 27, 2016
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.

3 participants