snap-confine

Merge lp://qastaging/~chipaca/snap-confine/tmpdir-simplify-lp1462916 into lp://qastaging/~snappy-dev/snap-confine/trunk

  1. tmpdir-simplify-lp1462916
  2. Merge into trunk
Proposed by John Lenton
Status: Merged
Approved by: Sergio Schvezov
Approved revision: 73
Merged at revision: 72
Proposed branch: lp://qastaging/~chipaca/snap-confine/tmpdir-simplify-lp1462916
Merge into: lp://qastaging/~snappy-dev/snap-confine/trunk
Diff against target: 167 lines (+42/-74)
4 files modified
debian/changelog (+7/-0)
debian/usr.bin.ubuntu-core-launcher (+3/-2)
src/main.c (+32/-38)
tests/test_tmpdir (+0/-34)
To merge this branch: bzr merge lp://qastaging/~chipaca/snap-confine/tmpdir-simplify-lp1462916
Reviewer Review Type Date Requested Status
Sergio Schvezov Approve
Tyler Hicks (community) Approve
Review via email: mp+261446@code.qastaging.launchpad.net

Commit message

Simplify TMPDIR handling.

Description of the change

This is lp:~mvo/ubuntu-core-launcher/tmpdir-simplify-lp1462916 with a strdup check.

To post a comment you must log in.
Revision history for this message
Tyler Hicks (tyhicks) wrote :

Looks good - thanks for taking care of the strdup checking.

review: Approve
Revision history for this message
Snappy Tarmac (snappydevtarmac) wrote :

Voting does not meet specified criteria. Required: Approve >= 1, Disapprove == 0. Got: 1 Pending.

Revision history for this message
Sergio Schvezov (sergiusens) :
review: Approve

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