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

kernel plugin: kernel targets depending on debarch #689

Merged

Conversation

sergiusens
Copy link
Collaborator

This allows using the same snapcraft.yaml to build kernels
for different arches as they have different image targets.

LP: #1604801

Signed-off-by: Sergio Schvezov sergio.schvezov@ubuntu.com

This allows using the same snapcraft.yaml to build kernels
for different arches as they have different image targets.

LP: #1604801

Signed-off-by: Sergio Schvezov <sergio.schvezov@ubuntu.com>
@sergiusens
Copy link
Collaborator Author

retest this please

@sergiusens
Copy link
Collaborator Author

@leannogasawara any updates on testing for this?

@leannogasawara
Copy link

On Wed, Aug 3, 2016 at 6:21 PM, Sergio Schvezov notifications@github.com
wrote:

@leannogasawara https://github.com/leannogasawara any updates on
testing for this?

Yep, Paolo tested this last week:

<goog_1295843>
https://bugs.launchpad.net/snapcraft/+bug/1604801/comments/3

"With this patch i was able to snap, from the same kernel tree, kernels for
amd64, arm64 and armhf - looks good."

Thanks,
Leann

@sergiusens
Copy link
Collaborator Author

thanks @leannogasawara :-)

@@ -23,8 +23,8 @@
The following kernel specific options are provided by this plugin:

- kernel-image-target:
(string; default: bzImage)
the kernel image make target to build; maps to make target.
(yaml object; default: bzImage)
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't this be: a string or a yaml object?
I think the help should explain that you can pass only one target, and how.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah probably so

'oneOf': [
{'type': 'string'},
{'type': 'object'},
],
Copy link
Contributor

Choose a reason for hiding this comment

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

I didn't know you could do this! Would have made some earlier changes easier.

@kyrofa
Copy link
Contributor

kyrofa commented Aug 5, 2016

I like it 👍

@sergiusens sergiusens merged commit f0a19eb into canonical:master Aug 5, 2016
@sergiusens sergiusens deleted the bugfix/1604801/kernel-target-selection branch August 30, 2016 16:58
kalikiana pushed a commit to kalikiana/snapcraft that referenced this pull request Apr 6, 2017
This allows using the same snapcraft.yaml to build kernels
for different arches as they have different image targets.

LP: #1604801

Signed-off-by: Sergio Schvezov <sergio.schvezov@ubuntu.com>
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.

None yet

4 participants