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

Automatically generate debian package with pkgr.io #6716

Merged
merged 2 commits into from
May 6, 2014
Merged

Conversation

crohr
Copy link
Contributor

@crohr crohr commented Apr 7, 2014

Here are the changes I had to make to generate debian packages of Gitlab for Ubuntu 12.04 and Debian 7.4, using https://pkgr.io. The installation process is currently described in https://pkgr.io/apps/pkgr/gitlabhq.

In addition to the specific stuff related to packaging, I added a new gitlab:shell:install to streamline the installation of gitlab-shell (with auto-generation of its config.yml file), and a way to set the Gitlab configuration using environment variables. Please let me know what you think.

@dosire
Copy link
Member

dosire commented Apr 7, 2014

Request to first merge commits 6cd937d (Allow to parse heroku-generated database.yml config file), and 6a80933 (Fix Procfile to be usable in production).

@@ -1,2 +1,2 @@
web: bundle exec unicorn_rails -p $PORT -E development -c config/unicorn_development.rb
web: bundle exec unicorn_rails -p ${PORT} -E ${RAILS_ENV} -c ${UNICORN_CONFIG:="config/unicorn.rb"}
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are this changes needed?

@jvanbaarsen
Copy link
Contributor

@dosire What about creating two seperate MR's for those fixes? That might clearify things a little

@dosire
Copy link
Member

dosire commented Apr 7, 2014

@jvanbaarsen I suggested creating separate MR's to @crohr but this was not done. I asked again.

@jvanbaarsen
Copy link
Contributor

@dosire Ok thanks :)

@crohr
Copy link
Contributor Author

crohr commented Apr 7, 2014

As requested, added two more PRs to split the fixes: #6718 and #6717.

@dosire
Copy link
Member

dosire commented Apr 7, 2014

@crohr Thanks for the split MR's!

@crohr
Copy link
Contributor Author

crohr commented Apr 15, 2014

We're really close. "Just" need #6782, #6783, and #6726 to have a fully working debian package. CentOS 6 support is also coming very soon.

@dosire
Copy link
Member

dosire commented Apr 22, 2014

Getting closer, only #6726 left for review by @randx

@crohr
Copy link
Contributor Author

crohr commented May 1, 2014

Rebased against latest master, and added support for Ubuntu 14.04 to the mix.

Also, packages are now automatically tested for regressions (see https://github.com/pkgr/showcase repository if you're curious).

Anything I can do to help the review on #6726 ?

@dosire
Copy link
Member

dosire commented May 2, 2014

@crohr Thanks for the update. Nothing you can do to help the review on #6726 for now, @randx told me this morning he would look into it today.

dosire added a commit that referenced this pull request May 6, 2014
Automatically generate debian package with pkgr.io
@dosire dosire merged commit aefdac0 into gitlabhq:master May 6, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants