postgresql*-server: chown: postgres: Invalid argument

[thehiddendepth@…]

When install postgresql83-server and postgresql90-server on Mac OS X 10.7.2, I have the following error:

Error: Target org.macports.destroot returned: shell command failed (see log for details)

And in the log file:

:debug:destroot Can't run destroot under sudo without elevated privileges (due to mtree).

[thehiddendepth@…]

postgresql90-server-main.log

[thehiddendepth@…]

postgresql83-server-main.log

[thehiddendepth@…]

Cc Me!

[ryandesign (Ryan Carsten Schmidt)]

The real error shown in both logs is:

chown: postgres: Invalid argument

Is there something wrong with the "postgres" user that these ports should have created? Does the user even exist?

[thehiddendepth@…]

Replying to ryandesign@…:

The real error shown in both logs is:

chown: postgres: Invalid argument

Is there something wrong with the "postgres" user that these ports should have created? Does the user even exist?

Hi, ryan,

thanks for the help. I am not sure if you are asking me. I checked on my MBP, there's no user: postgres created, but there's a group 'postgres' created.

Best Regards. Larry

[drkp (Dan Ports)]

Looks like this is happening because the port is setting add_users in the destroot phase, which doesn't work right. It should do this outside the phase (add_users is a variable that gets processed at the appropriate time, not a command like adduser). The attached patch fixes this, and also lets add_users handle adding the group instead of using addgroup. This fixes the problem for me, and I believe it will do the right thing even when using binary archives.

[jeremyhu (Jeremy Huddleston Sequoia)]

@andholt, since you dropped off IRC:

<andholt> can anyone please help me figure out how to apply a patch to a portfile?
<andholt> running into this problem: https://trac.macports.org/ticket/33497
* andholt has quit (Client Quit)
<jeremyhu> andholt: cd /path/tpo/dports/databases/postgresql91-server ; curl -L https://trac.macports.org/raw-attachment/ticket/33497/postgresql91-server.patch | patch

[drkp (Dan Ports)]

I committed this for postgresql91-server in r91170. Still need to fix the other versions...

[drkp (Dan Ports)]

...and for 83, 84, and 90 in r91171. The earlier versions still use adduser, but they're so old they should probably be removed anyway...