Skip to content

Add dist to changelog#76

Open
connortechnology wants to merge 5 commits intopackpack:masterfrom
connortechnology:add_dist_to_changelog
Open

Add dist to changelog#76
connortechnology wants to merge 5 commits intopackpack:masterfrom
connortechnology:add_dist_to_changelog

Conversation

@connortechnology
Copy link
Copy Markdown

So we are trying to use packpack to generate packages and then use those packages with mini-dinstall to make a ppa for our nightly builds.

Problem is that packpack doesn't set the distro field in the changelog, so it defaults to UNRELEASED which isn't very useful.

This PR adds a variable CHANGELOG_DIST and defaults it to $DIST and uses it on the dch command line to set the distro value.

@knight-of-ni
Copy link
Copy Markdown
Contributor

This pr is a result of our work on the ZoneMinder project. I encouraged @connortechnology to engage the packpack community, since his questions were beyond my current experience with debian packages.

@gdevenyi
Copy link
Copy Markdown
Contributor

gdevenyi commented Jun 9, 2017

Looks like this is breaking the Traivs check because DIST isn't being properly supplied into the build.

@connortechnology
Copy link
Copy Markdown
Author

I havn't been able to figure out any way to use $(DIST) in the makefiles. It just isn't populated. But you can populate CHANGELOG_DIST and that seems to work.

@connortechnology
Copy link
Copy Markdown
Author

This is not breaking travis anymore.

@knight-of-ni
Copy link
Copy Markdown
Contributor

Are there any objections to merging this? This would be helpful for the ZoneMinder project.

Update from upstream master
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