Opened 8 years ago
Last modified 8 years ago
#51671 new update
remctl @3.11 - Updated portfile (replacing version 3.1)
| Reported by: | akkornel (A. Karl Kornel) | Owned by: | quentinmit (Quentin Smith) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | Cc: | ||
| Port: | remctl |
Description
Hello!
I have an updated Portfile for the remctl port. The current Portfile in MacPorts is for version 3.1; the latest version is 3.11.
The first big difference between 3.1 and 3.11 is that libevent is required, so that port has been added to the depends list. Also, paths to pkg-config, prce-config, and krb5-config have been explicitly set.
Another big thing: --enable-reduced-depends has been removed, because it causes remctl to build against the system Kerberos/GSSAPI, which does not work. A note to this effect has been added to the Portfile.
Since remctl also has an optional daemon component (like OpenSSH does), I've added a startupitem to the Portfile.
One thing that could be done in the future is to add support for building Perl, PHP, Python, and Ruby. The upstream distribution already has the code to build the bindings; it just needs to be integrated into the Portfile somehow.
Please let me know if you have any questions or concerns about the Portfile. Thanks very much!
Attachments (3)
Change History (7)
Changed 8 years ago by akkornel (A. Karl Kornel)
comment:1 follow-up: 2 Changed 8 years ago by mf2k (Frank Schima)
| Owner: | changed from macports-tickets@… to quentin@… |
|---|---|
| Type: | submission → update |
Thanks. A "submission" ticket type is only for submitting a new port. Per the guidelines, please instead attach a unified diff of the Portfile so we can easily see what changes you are proposing.
In the future, please Cc the port maintainers (port info --maintainers remctl), if any.
Changed 8 years ago by akkornel (A. Karl Kornel)
| Attachment: | Portfile.diff added |
|---|
Diff from existing Portfile
Changed 8 years ago by akkornel (A. Karl Kornel)
| Attachment: | Portfile.2 added |
|---|
Portfile for remctl 3.11
comment:2 Changed 8 years ago by akkornel (A. Karl Kornel)
Replying to mf2k@…:
Thanks. A "submission" ticket type is only for submitting a new port. Per the guidelines, please instead attach a unified diff of the Portfile so we can easily see what changes you are proposing.
OK, will do! I've attached a diff, and I've also attached an updated Portfile (attached as "Portfile.2") that fixes the Subversion Id keyword substitution.
In the future, please Cc the port maintainers (
port info --maintainers remctl), if any.
OK. I see you've sent this over to Quentin. Thanks much!
comment:3 follow-up: 4 Changed 8 years ago by mf2k (Frank Schima)
Comments:
- The
revisionline should be deleted because it starts at 0 when increasing the version and that is the default value. - You have changed the maintainer to you. Is this intentional? Have you discussed this with the current maintainer? Normally this type of action is handled separately.
- You added the line
use_configure yesand that is the default value. So it should be removed.
comment:4 Changed 8 years ago by quentinmit (Quentin Smith)
Hi! Let me take a look.
Replying to mf2k@…:
- You have changed the maintainer to you. Is this intentional? Have you discussed this with the current maintainer? Normally this type of action is handled separately.
He has not discussed this with me.
Re: the diff:
- It definitely can link against the system Kerberos, I have seen that work. El Capitan has a pretty crippled system Kerberos, however, so I can definitely believe it doesn't work now. What platforms did you test this on?
- The startup item should use startupitem.executable, not startupitem.start. You are passing -F which will cause remctld to not detach from launchd; launchd is perfectly capable of tracking the process lifetime, without using a pid file.
Portfile for remctl 3.11