-
Notifications
You must be signed in to change notification settings - Fork 17
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 #84
base: master
Are you sure you want to change the base?
Conversation
CI job passed. |
CI job failed. Please visit http://lists.gpii.net/pipermail/ci/ for more details. |
ok to test |
CI job passed. |
CI job failed. Please visit http://lists.gpii.net/pipermail/ci/ for more details. |
CI job passed. |
url: pkgdata.homepage, | ||
buildArch: currentArch, | ||
requires: [ | ||
"nodejs >= 0.10.42", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we make this >= 6.9.2
just to ensure people are using a modern environment?
|
||
To build a RPM package run the following command at the root of the git repository: | ||
|
||
grunt build-rpm-docker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to run npm install grunt-easy-rpm
before this command would work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm getting the following error:
cp: cannot access '/sync': Permission denied
grunt-cli: The grunt command line interface (v1.2.0)
Fatal error: Unable to find local grunt.
If you're seeing this message, grunt hasn't been installed locally to
your project. For more information about installing and configuring grunt,
please see the Getting Started guide:
http://gruntjs.com/getting-started
cp: cannot stat '/packages/bin': No such file or directory
Warning: Command failed: docker run --rm -i -v $(pwd):/sync fedora /bin/bash -c 'dnf install -y nodejs-grunt-cli; cp -r /sync /packages; cd /packages; grunt build-rpm; cp -r /packages/bin /sync'
Unable to find image 'fedora:latest' locally
Trying to pull repository docker.io/library/fedora ...
sha256:fbeff4f6ec6ce2393c77aff9b34e19e8a9e355fd46664613f14a36394adf8e71: Pulling from docker.io/library/fedora
5b0bacb7828e: Pulling fs layer
5b0bacb7828e: Download complete
5b0bacb7828e: Pull complete
Digest: sha256:fbeff4f6ec6ce2393c77aff9b34e19e8a9e355fd46664613f14a36394adf8e71
Status: Downloaded newer image for docker.io/fedora:latest
warning: /var/cache/dnf/fedora-310f9d37d74ceec1/packages/nodejs-grunt-cli-1.2.0-1.fc25.noarch.rpm: Header V3 RSA/SHA256 Signature, key ID fdb19c98: NOKEY
Importing GPG key 0xFDB19C98:
Userid : "Fedora 25 Primary (25) <[email protected]>"
Fingerprint: C437 DCCD 558A 66A3 7D6F 4372 4089 D8F2 FDB1 9C98
From : /etc/pki/rpm-gpg/RPM-GPG-KEY-fedora-25-x86_64
cp: cannot access '/sync': Permission denied
cp: cannot stat '/packages/bin': No such file or directory
Use --force to continue.
Aborted due to warnings.
Full output: https://gist.github.com/gtirloni/6edd489d441f476869cb2638fdabd250
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems like the directories /sync and /packages don't exist inside the official fedora image.
I'm wondering if basing this work on our RPM builder image would be better (https://github.com/idi-ops/docker-centos-rpm-builder), but I guess that would be built against CentOS and we're targeting Fedora desktop packages here, right? Maybe this affords building a docker-fedora-rpm-builder
image too.
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