Opened 11 years ago

Closed 10 years ago

#23405 closed enhancement (fixed)

openldap: allow non-root install

Reported by: cgilmour@… Owned by: landonf (Landon Fuller)
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc: ryandesign (Ryan Schmidt)
Port: openldap

Description

Hi,

I was trying to compile Macports as a regular user (instead of running via sudo), and an issue occurred with openldap. Similar in concept to the dbus "no_root" variant, I modified the openldap port file to allow this.

Thanks, Caleb

Attachments (1)

openldap.Portfile.trac23405_v1.diff (1.1 KB) - added by cgilmour@… 11 years ago.
Diff for adding "no_root" variant

Download all attachments as: .zip

Change History (5)

Changed 11 years ago by cgilmour@…

Diff for adding "no_root" variant

comment:1 Changed 11 years ago by cgilmour@…

Changes are:

  1. Add variant, and set startupitem.create to no. If it stays set to yes, build would fail when "activating", as it applies the other startup parameters
  2. Wrap the group creation with a 'variant isset' check
  3. Wrap the xinstall command with a 'variant isset' check

comment:2 Changed 11 years ago by jmroot (Joshua Root)

Keywords: haspatch added
Owner: changed from macports-tickets@… to landonf@…
Version: 1.8.2

Please remember to cc the maintainer.

comment:3 Changed 11 years ago by ryandesign (Ryan Schmidt)

Cc: ryandesign@… added

I'm not thrilled with adding a variant for this purpose.

Instead, it would make sense for the user to set startupitem_type to none in macports.conf when he installs MacPorts as a non-root user and does not want launchd plists created. Then, the port could be modified to only create the user account and data directory if ${startupitem.type} is not none. At least I theorize this should work; I have not tested.

comment:4 Changed 10 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: newclosed
Summary: openldap no_root variantopenldap: allow non-root install

Base changes make most of this unnecessary. Fixed the one thing that would have still been preventing non-root installation in r76807.

Note: See TracTickets for help on using tickets.