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

ncm-spma: partial package version resolution + include of config-common #174

Merged
merged 2 commits into from
Mar 29, 2014

Conversation

jouvin
Copy link
Contributor

@jouvin jouvin commented Mar 20, 2014

Includes reverting #156

@jouvin jouvin changed the title ncm-spma: partial package version resolution + cosmetic changes ncm-spma: partial package version resolution + include of config-common Mar 21, 2014
@piojo-zz
Copy link
Member

I guess we should discuss this one in the next stand-up meeting?

@jouvin
Copy link
Contributor Author

jouvin commented Mar 28, 2014

Ok, I keep it open. But most of you failed to join this week and I will not be able to join next one...

Michel

@ned21
Copy link
Contributor

ned21 commented Mar 28, 2014

Only commits a829084 and a826e0c need discussion and I think we agreed the right way to fix those in #169. So this PR can be merged if those commits are removed or replaced by what was agreed in #169.

@jouvin
Copy link
Contributor Author

jouvin commented Mar 29, 2014

I'll implement #169 and update this PR accordingly.

…sistency

- config.pan as the main entry point for all platforms
- variable CONFIG_MODULES_CONFIG_SUFFIX to choose flavor
@jouvin
Copy link
Contributor Author

jouvin commented Mar 29, 2014

Should be ok now... if Jenkins is happy!

ned21 added a commit that referenced this pull request Mar 29, 2014
ncm-spma: partial package version resolution + include of config-common
@ned21 ned21 merged commit 0a09f5e into quattor:master Mar 29, 2014
aka7 pushed a commit to aka7/configuration-modules-core that referenced this pull request Oct 14, 2022
functions filesystem: partitions_add: fix number of required arguments bug
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