Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

Conversation

@iaguis
Copy link
Member

@iaguis iaguis commented Jan 13, 2016

We don't include libacl in CoreOS so this broke rkt there. Reverting.

Fixes #1961

@iaguis iaguis force-pushed the iaguis/revert-acl branch from 4a765ff to fb3948c Compare January 13, 2016 16:32
@iaguis
Copy link
Member Author

iaguis commented Jan 14, 2016

Not needed anymore, the new rkt ebuild depends on libacl: coreos/coreos-overlay#1714. Closing.

@iaguis iaguis closed this Jan 14, 2016
@alban
Copy link
Member

alban commented Jan 14, 2016

The run-time requirements are not up-to-date then: it does not list libacl:
https://github.com/coreos/rkt/blob/master/Documentation/hacking.md#run-time-requirements

@iaguis
Copy link
Member Author

iaguis commented Jan 14, 2016

The run-time requirements are not up-to-date then: it does not list libacl:
https://github.com/coreos/rkt/blob/master/Documentation/hacking.md#run-time-requirements

Right.

If we decide we want #1963 or something similar, it will not be required. If we decide we don't, we should update the docs.

@jonboulle
Copy link
Contributor

Let's update the docs right now, then if we decide we want #1963 we can
remove it from them again.

On Thu, Jan 14, 2016 at 11:07 AM, Iago López Galeiras <
[email protected]> wrote:

The run-time requirements are not up-to-date then: it does not list libacl:

https://github.com/coreos/rkt/blob/master/Documentation/hacking.md#run-time-requirements

Right.

If we decide we want #1963 #1963 or
something similar, it will not be required. If we decide we don't, we
should update the docs.


Reply to this email directly or view it on GitHub
#1965 (comment).

@iaguis
Copy link
Member Author

iaguis commented Jan 14, 2016

Let's update the docs right now, then if we decide we want #1963 we can
remove it from them again.

👍

@iaguis
Copy link
Member Author

iaguis commented Jan 14, 2016

#1975

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants