490130 – app-admin/puppet with USE=minimal doesn't seem to need hiera

Bug 490130 - app-admin/puppet with USE=minimal doesn't seem to need hiera

Summary: app-admin/puppet with USE=minimal doesn't seem to need hiera

Status:	RESOLVED INVALID

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Sysadmin Bugs

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2013-11-02 12:50 UTC by Theo Chatzimichos (RETIRED)
Modified:	2013-11-07 20:01 UTC (History)
CC List:	2 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Theo Chatzimichos (RETIRED) archtester

2013-11-02 12:50:53 UTC

If my tests are right, hiera is needed only in the master. Thus, if there are no objections within a week, I'll commit the following:

@@ -18,12 +18,12 @@
 IUSE="augeas diff doc emacs ldap minimal rrdtool selinux shadow sqlite3 vim-syntax xemacs"
 
 ruby_add_rdepend "
-       dev-ruby/hiera
        >=dev-ruby/facter-1.6.2
        augeas? ( dev-ruby/ruby-augeas )
        diff? ( dev-ruby/diff-lcs )
        doc? ( dev-ruby/rdoc )
        ldap? ( dev-ruby/ruby-ldap )
+       !minimal? ( dev-ruby/hiera )
        shadow? ( dev-ruby/ruby-shadow )
        sqlite3? ( dev-ruby/sqlite3 )
        virtual/ruby-ssl"

Comment 1 Theo Chatzimichos (RETIRED) archtester

2013-11-07 20:01:34 UTC

and yes i was wrong, it is needed for `puppet apply`, thanks once again to Adrien for being such a responsive upstream