Add capability hook messages #310

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
Contributor

zyga commented Jan 11, 2016

NOTE: this is meant to start the discussion on the hook system. Hooks are further described in the system design document.

This patch adds the set of messages used by the capability hook
protocol. Each message is accompanied by a small helper used for
creating messages of this type.

If the idea is approved I will continue with additional code for running a hook and transition current capability code to use hooks from snaps that contain the appropriate meta-data and hook executable.

Add capability hook messages
This patch adds the set of messages used by the capability hook
protocol. Each message is accompanied by a small helper used for
creating messages of this type.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
+// -*- Mode: Go; indent-tabs-mode: t -*-
+
+/*
+ * Copyright (C) 2015 Canonical Ltd
@stevenwilkin

stevenwilkin Jan 11, 2016

Member

I'd now 2016 ;)

@zyga

zyga Jan 11, 2016

Contributor

Heh, thanks :) Will fix :)

2016-01-11 12:31 GMT+01:00 Steven Wilkin notifications@github.com:

In caps/hooks.go
ubuntu-core#310 (comment):

@@ -0,0 +1,283 @@
+// -- Mode: Go; indent-tabs-mode: t --
+
+/*

  • * Copyright (C) 2015 Canonical Ltd

I'd now 2016 ;)


Reply to this email directly or view it on GitHub
https://github.com/ubuntu-core/snappy/pull/310/files#r49311610.

Contributor

niemeyer commented Jan 11, 2016

I'll close this for the time being to prevent others from spending much time on it before we have some more agreement on the design. This is not a rejection.. I've read it and would like to talk through the next steps before we jump into this code.

@niemeyer niemeyer closed this Jan 11, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment