Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 91729 - PostgreSQL ebuild - Remove sudo dependence
Summary: PostgreSQL ebuild - Remove sudo dependence
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: PgSQL Bugs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-05-06 12:47 UTC by Stephen Leaf
Modified: 2007-09-22 23:28 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stephen Leaf 2005-05-06 12:47:28 UTC
--- /usr/portage/dev-db/postgresql/postgresql-8.0.1-r2.ebuild   2005-05-01 03:05:25.000000000 -0500
+++ postgresql-8.0.1-r2.ebuild  2005-05-06 14:41:38.000000000 -0500
@@ -34,7 +34,6 @@
        nls? ( sys-devel/gettext )
        kerberos? ( virtual/krb5 )"
 RDEPEND="virtual/libc
-       app-admin/sudo
        zlib? ( >=sys-libs/zlib-1.1.3 )
        tcltk? ( >=dev-lang/tcl-8 )
        perl? ( >=dev-lang/perl-5.6.1-r2 )
@@ -205,7 +204,7 @@
                        eerror "Temporary setting this value to ${SEMMNI_MIN} while creating the initial database."
                        echo ${SEM} ${SEMMNI_MIN} > /proc/sys/kernel/sem
                fi
-               sudo -u postgres /usr/bin/initdb --pgdata ${PG_DIR}/data
+               su postgres -c "/usr/bin/initdb --pgdata ${PG_DIR}/data"

                if [ ! `sysctl -n kernel.sem | cut -f4` -eq ${SEMMNI} ] ; then
                        echo ${SEM} ${SEMMNI} > /proc/sys/kernel/sem


The above patch removes the dependence on sudo. Was there any reason to have a dependence on sudo in the first place?

Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Comment 1 Masatomo Nakano (RETIRED) gentoo-dev 2005-05-08 14:26:11 UTC
The only reason is that I forgot to remove sudo dependency from postgresql-8* when I removed it from postgresql-7*.

I've just removed the dependency from unstable ebuilds.

Thanks for reporting.