kernel plugin: rework MAKEFLAGS from the environment #1150

Merged
merged 4 commits into from Mar 22, 2017

Conversation

Projects
None yet
2 participants
Contributor

piso77 commented Feb 17, 2017

Linux's kernel Makefile gets confused if it is invoked with the
environment already setup by another Linux's Makefile:

linux/package/Makefile -> snapcraft -> linux/Makefile

fix the problem removing the offending make variable (MAKEFLAGS)

Signed-off-by: Paolo Pisati paolo.pisati@canonical.com

LP: #1665661

thanks for this, mind looking at the inline comment and updating the unit tests for this?

snapcraft/plugins/kbuild.py
# build the software
- self.run(self.make_cmd + self.make_targets)
+ self.run(self.make_cmd + self.make_targets, env=env)
@sergiusens

sergiusens Feb 21, 2017

Collaborator

can you restore the variable after this call?

kernel plugin: rework MAKEFLAGS from the environment
Signed-off-by: Paolo Pisati <paolo.pisati@canonical.com>

@piso77 piso77 changed the title from kernel plugin: remove MAKEFLAGS from the environment to kernel plugin: rework MAKEFLAGS from the environment Mar 21, 2017

@sergiusens sergiusens merged commit 1409ed2 into snapcore:master Mar 22, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@piso77 piso77 deleted the piso77:2.27_fixenv branch Mar 24, 2017

elopio added a commit that referenced this pull request Mar 27, 2017

kernel plugin: rework MAKEFLAGS from the environment (#1150)
LP: #1665661
Signed-off-by: Paolo Pisati <paolo.pisati@canonical.com>

kalikiana pushed a commit to kalikiana/snapcraft that referenced this pull request Apr 6, 2017

kernel plugin: rework MAKEFLAGS from the environment (#1150)
LP: #1665661
Signed-off-by: Paolo Pisati <paolo.pisati@canonical.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment