Skip to content

[4.6/4.6.1] IMPORTANT for 4.6.1 release -- debian: allow rules to pick ACS_BUILD_OPTS from env#1149

Closed
yadvr wants to merge 1 commit into
apache:masterfrom
shapeblue:4.6-debian-noredist-fix
Closed

[4.6/4.6.1] IMPORTANT for 4.6.1 release -- debian: allow rules to pick ACS_BUILD_OPTS from env#1149
yadvr wants to merge 1 commit into
apache:masterfrom
shapeblue:4.6-debian-noredist-fix

Conversation

@yadvr
Copy link
Copy Markdown
Member

@yadvr yadvr commented Dec 2, 2015

Only now debian builds can be noredist etc.

cc @remibergsma @wido - I think the apt-get.eu 4.6.0 deb repo was oss and not noredist one. I also checked the jenkins job it does not "export ACS_BUILD_OPTS='-Dnoredist -Dnonoss'" before running dpkg-buildpackage

I discovered this while testing 4.6.1 RC1 today, nothing major just a build profile issue.

Only now debian builds can be noredist etc.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@yadvr yadvr changed the title debian: allow rules to pick ACS_BUILD_OPTS from env [4.6/4.6.1] IMPORTANT for 4.6.1 release -- debian: allow rules to pick ACS_BUILD_OPTS from env Dec 2, 2015
@yadvr
Copy link
Copy Markdown
Member Author

yadvr commented Dec 2, 2015

@remibergsma
Copy link
Copy Markdown
Contributor

Makes sense, LGTM also based on the successful job above.

@remibergsma
Copy link
Copy Markdown
Contributor

Ping @borisroman to have a look

@wido
Copy link
Copy Markdown
Contributor

wido commented Dec 3, 2015

Seems good, LGTM

@remibergsma
Copy link
Copy Markdown
Contributor

@bhaisaab Should I merge this to master, or should it be merged to 4.6?

@yadvr
Copy link
Copy Markdown
Member Author

yadvr commented Dec 4, 2015

@remibergsma sorry for the source/target branches; this needs to go into both 4.6 and master branches. So merge on 4.6, and fwd-merge 4.6 on master. Thanks.

@yadvr
Copy link
Copy Markdown
Member Author

yadvr commented Dec 4, 2015

@remibergsma let me know if you want me to open a new PR against 4.6 branch.

@remibergsma
Copy link
Copy Markdown
Contributor

@bhaisaab Check! This I cannot merge to 4.6 with this PR so if you make one against 4.6 that would be easy. Mention this one so it can go through faster. Thanks!

@yadvr
Copy link
Copy Markdown
Member Author

yadvr commented Dec 4, 2015

@remibergsma thanks, opened another PR #1169 please merge that. Closing this one, as once the other PR will be merged on 4.6 and 4.6 on master; we won't need this PR anyway.

@yadvr yadvr closed this Dec 4, 2015
asfgit pushed a commit that referenced this pull request Dec 4, 2015
debian: allow rules to pick ACS_BUILD_OPTS from envOnly now debian builds can be noredist etc.

Old PR: #1149

cc @remibergsma

* pr/1169:
  debian: allow rules to pick ACS_BUILD_OPTS from env

Signed-off-by: Remi Bergsma <github@remi.nl>
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