Opened 11 years ago

Closed 11 years ago

#41897 closed update (fixed)

argus-clients @3.0.6 port update

Reported by: todd@… Owned by: usx303@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: argus-clients

Description

The attached diff updates argus-clients to version 3.0.6. I also made SASL inclusion a default variant so it can be disabled if desired. Tested on OS X 10.6.8 with xcode 3.2.6 and OS X 10.9.1 with xcode 5.0.2.

Attachments (1)

Portfile-argus-clients.diff (1.7 KB) - added by todd@… 11 years ago.
updated diff

Download all attachments as: .zip

Change History (6)

comment:1 Changed 11 years ago by mf2k (Frank Schima)

Cc: usx303@… openmaintainer@… removed
Owner: changed from macports-tickets@… to usx303@…
Version: 2.2.1

Thanks. Note the following however:

  • openmaintainer is not a valid email, please do not Cc that address.
  • The revision line needs to be removed as with all version upgrades.
  • A sha256 checksum should be added, and the md5 and sha1 checksums can be removed unless upstream provides those for verification.
  • Why do you create a variant for sasl? Is there some reason not to always have cyrus-sasl compatibility?

Changed 11 years ago by todd@…

Attachment: Portfile-argus-clients.diff added

updated diff

comment:2 in reply to:  1 Changed 11 years ago by todd@…

Sorry for the newb errors with emails, etc. I have updated the patch without the revision line, and included a sha256. The upstream does provide a md5.

I created a variant for sasl because the documentation from the project doesn't really address sasl configuration for just getting up and running. For example, using the sasl enabled argus/argus-clients from macports and following the directions for setting up probes and clients from the project wiki (http://nsmwiki.org/index.php?title=Argus) results in a SASL error, because SASL configuration is not covered. For someone wanting to play with it for a bit before getting into SASL configuration, disabling the SASL option results in a working install. SASL is still enabled by default, but can be optionally disabled. Just my opinion though, so if you want it the way it was (no variant) then I can do that too.

comment:3 Changed 11 years ago by mf2k (Frank Schima)

Thanks for the updates. Let's wait for the maintainer to respond about the sasl variant. If there is no response in 3 days, we can commit it with the variant.

comment:4 Changed 11 years ago by todd@…

Bump. No response from the maintainer for 13 days.

comment:5 Changed 11 years ago by mf2k (Frank Schima)

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.