/juju

Make all juju/cert paths point now to utils/cert #6556

Merged
merged 4 commits into from Nov 11, 2016

Conversation

Reviewers

@perrito666 perrito666

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@hoenirvili @jujubot @perrito666
@hoenirvili
Contributor

hoenirvili commented Nov 10, 2016

This patch depends on this patch to be merged juju/utils#250

This patch contains:

  • moves juju/cert into utils/ pkg.
  • update code path for all files.
  • update dependencies.tsv
Make all juju/cert paths point now to utils/cert 
Signed-off-by: Salvatore Giulitti <sgiulitti@cloudbasesolutions.com>
No user is associated with the committer email.
Learn about signing commits
222a41f

Salvatore Giulitti added some commits Nov 10, 2016

Changes after review from juju/utils. 
Signed-off-by: Salvatore Giulitti <sgiulitti@cloudbasesolutions.com>
No user is associated with the committer email.
Learn about signing commits
ac1ae3d
Try and fix some build issues. 
Signed-off-by: Salvatore Giulitti <sgiulitti@cloudbasesolutions.com>
No user is associated with the committer email.
Learn about signing commits
6aec75a
More fixes. 
Signed-off-by: Salvatore Giulitti <sgiulitti@cloudbasesolutions.com>
No user is associated with the committer email.
Learn about signing commits
baecf5e
@hoenirvili
Contributor

hoenirvili commented Nov 11, 2016 •
Edited 1 time

$$merge$$
@jujubot
Contributor

jujubot commented Nov 11, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit a902b41 into juju:develop Nov 11, 2016
1 check passed

1 check passed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details

@hoenirvili hoenirvili deleted the hoenirvili:move-cert-pkg branch Nov 11, 2016

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