Merge lp://qastaging/~sergiusens/snapcraft/mono into lp://qastaging/~snappy-dev/snapcraft/core

Proposed by Sergio Schvezov
Status: Needs review
Proposed branch: lp://qastaging/~sergiusens/snapcraft/mono
Merge into: lp://qastaging/~snappy-dev/snapcraft/core
Diff against target: 169 lines (+128/-0)
8 files modified
examples/mono-project/hello-google.cs (+29/-0)
examples/mono-project/meta/package.yaml (+6/-0)
examples/mono-project/meta/readme.md (+1/-0)
examples/mono-project/snapcraft.yaml (+9/-0)
plugins/mono.yaml (+3/-0)
plugins/mono_project.yaml (+11/-0)
snapcraft/plugins/mono.py (+34/-0)
snapcraft/plugins/mono_project.py (+35/-0)
To merge this branch: bzr merge lp://qastaging/~sergiusens/snapcraft/mono
Reviewer Review Type Date Requested Status
Ricardo Salveti Pending
Michael Vogt Pending
Jamie Strandboge Pending
Snappy Developers Pending
Review via email: mp+267894@code.qastaging.launchpad.net

Commit message

Simple mono project

Description of the change

This is a practice MP that can be turned into a mergeable one.

The current example builds fine, but fails to run due to:
[27737.707903] audit: type=1400 audit(1439434887.243:723): apparmor="DENIED" operation="mknod" profile="mono-hello-google.sideload_run_0.1" name="/dev/shm/mono.2131" pid=2131 comm="hello-google" requested_mask="c" denied_mask="c" fsuid=1000 ouid=1000

Not sure if a general apparmor rule is warranted:

  /dev/shm/mono.** rmw, # adding pid is desired.

I'm also not sure how to reuse assembly parts as that is tied to cmd (common.env) and does not allow me to easily -r:[assembly] .. IOW, setting the env (involving knowledge of the other parts) from the plugin would be nice.

A simple hello world works fine.

To post a comment you must log in.
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

If we were to add a rule, we could use this:

  /dev/shm/mono.${pid} rw,

${pid} today just expands to something approaching [0-9]* but the plan is to some day have kernel side variables such that ${pid} would correspond to the running process.

That said, we already have these rules:
/{dev,run}/shm/snaps/@{APP_PKGNAME}/ r,
/{dev,run}/shm/snaps/@{APP_PKGNAME}/** rk,
/{dev,run}/shm/snaps/@{APP_PKGNAME}/@{APP_VERSION}/ r,
/{dev,run}/shm/snaps/@{APP_PKGNAME}/@{APP_VERSION}/** mrwlkix,

Is there an environment variable that can be set in the launcher so that mono will just put the file in /dev/shm/snaps/@{APP_PKGNAME}/@{APP_VERSION}/ ?

Unmerged revisions

132. By Sergio Schvezov

Simple mono project

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches