Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#57765 closed submission (fixed)

pdsh and genders

Reported by: unix-fan Owned by: mf2k (Frank Schima)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: jtrammel
Port: genders pdsh

Description

hi team, if you're interested, i created an updated portfile for pdsh, and added support for genders. keep up the great work, still prefer macports over brew :) best regards, martin

Attachments (9)

Portfile.genders.txt (2.0 KB) - added by unix-fan 6 years ago.
Portfile.genders.2.txt (2.0 KB) - added by unix-fan 6 years ago.
Portfile.pdsh.txt (2.3 KB) - added by unix-fan 6 years ago.
Portfile.genders.3.txt (1.3 KB) - added by unix-fan 6 years ago.
Portfile.pdsh.2.txt (1.7 KB) - added by unix-fan 6 years ago.
Portfile.genders.4.txt (1.3 KB) - added by unix-fan 6 years ago.
undid whitespace changes
Portfile.pdsh.2.diff (2.4 KB) - added by unix-fan 6 years ago.
further cleanup
Portfile.pdsh.3.diff (1.9 KB) - added by unix-fan 6 years ago.
Portfile.pdsh.diff (1.9 KB) - added by unix-fan 6 years ago.

Download all attachments as: .zip

Change History (28)

Changed 6 years ago by unix-fan

Attachment: Portfile.genders.txt added

Changed 6 years ago by unix-fan

Attachment: Portfile.genders.2.txt added

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

Keywords: net pdsh genders removed
Port: genders added; net-pdsh net-genders removed

comment:2 Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)

Thanks.

You mentioned an updated pdsh port but I don't see anything attached about that.

In your new port for genders, please:

  • Add your GitHub handle to the maintainers line in the usual format.
  • Remove the homepage line; the github.setup line sets that for you.
  • Remove the master_sites and livecheck.* lines; use github.tarball_from releases instead.
  • Remove the comment lines
  • Add size to checksums
  • Fix the github.setup line. The correct value may be github.setup chaos genders 1-22-1 genders-; then you'll need to fix the version, possibly by setting it to a munged version of ${github.version} where you replace the dashes with periods.

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

I see now that the tag name is genders-1-22-1 but the filename is genders-1.22.tar.gz, so that's confusing and not anticipated by the github portgroup and will require some manual corrections in the portfile.

Changed 6 years ago by unix-fan

Attachment: Portfile.pdsh.txt added

Changed 6 years ago by unix-fan

Attachment: Portfile.genders.3.txt added

Changed 6 years ago by unix-fan

Attachment: Portfile.pdsh.2.txt added

comment:4 Changed 6 years ago by unix-fan

cleaned up Portfile.pdsh.txt.2 and Portfile.genders.txt.3 sorry, i'm not fit enough in macporting to get it to work without master_sites and the livecheck stuff.

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

There is already an existing pdsh port. Please instead modify that to update to 2.33 and attach a unified diff of the Portfile.

Changed 6 years ago by unix-fan

Attachment: Portfile.genders.4.txt added

undid whitespace changes

comment:6 Changed 6 years ago by mf2k (Frank Schima)

Cc: james.trammell@… added
Port: pdsh added

You have removed the previous maintainer from pdsh.

@ james.trammell: Are you OK with that?

comment:7 Changed 6 years ago by unix-fan

sorry, wasnt aware you can have more than one maintainer. i sent mail to james.trammell some time ago, didn't answer though. all this just to have the latest pdsh in macports :)

comment:8 Changed 6 years ago by mf2k (Frank Schima)

Well we don't want to step on anyone's toes. Although it does look like this maintainer is inactive. Last activity was 2015 when the port was created with him as maintainer.

comment:9 Changed 6 years ago by mf2k (Frank Schima)

Some comments on your pdsh patch:

  • You should be able to remove the master_sites line by setting github.tarball_from releases
  • There are too many variants being added IMHO. Can they all just be incorporated directly and not make them variants at all?
  • For the no_rsh variant, we long ago excised such "negative" variants for consistency. You made it the default, so is there a reason not to have rsh functionality built? If so, name the variant rsh and do not make it a default variant.
Last edited 6 years ago by mf2k (Frank Schima) (previous) (diff)

Changed 6 years ago by unix-fan

Attachment: Portfile.pdsh.2.diff added

further cleanup

comment:10 Changed 6 years ago by mf2k (Frank Schima)

Just to be clear. I want the variants removed entirely. Is there an actual reason to not just always have the functionality turned on? With the exception of the readline functionality, no other dependencies are being added.

Last edited 6 years ago by mf2k (Frank Schima) (previous) (diff)

Changed 6 years ago by unix-fan

Attachment: Portfile.pdsh.3.diff added

comment:11 Changed 6 years ago by unix-fan

removed variant rsh, doesnt make much sense anyway, unless you want talk to really old gear.

removed nodeupdown, because i couldn't find an uptodate source repo for it. thought it would be nice for experimenting, but not worth the effort. removed readline option, made it fix, because everyone wants to have it anyway. removed dsh option, made it fix, because it doesnt cost anything to have it supported.

would like to keep genders as an option for fiddling around, because it causes a conflict, when used together with dshgroups, and needs unexpected manual interaction. if it really bothers someone to keep it in there, then no, remove it for gods sake. then you also can ignore the genders portfile, of course.

:)

Changed 6 years ago by unix-fan

Attachment: Portfile.pdsh.diff added

comment:12 Changed 6 years ago by mf2k (Frank Schima)

Looking better, thank you! Do you guys mind if we add openmaintainer to this port too?

comment:13 Changed 6 years ago by mf2k (Frank Schima)

BTW, the genders variant is fine to keep.

Last edited 6 years ago by mf2k (Frank Schima) (previous) (diff)

comment:14 Changed 6 years ago by mf2k (Frank Schima)

I have filed #57791 for James. If I don't hear from him in 3 days, I'll commit this with unix-fan as maintainer with openmaintainer.

comment:15 Changed 6 years ago by jtrammel

Acknowleged. Thanks, I'm caught up with this thread now.

comment:16 Changed 6 years ago by mf2k (Frank Schima)

@jtrammel: Thank you for the reply! I'm happy to keep you as co-maintainer for this port. I will also update for GitHub handle on the port.

comment:17 Changed 6 years ago by mf2k (Frank Schima)

comment:18 Changed 6 years ago by mf2k (Frank Schima)

Owner: set to mf2k
Resolution: fixed
Status: newclosed

In 7d9c6c5a21fd3a09f73b29d4df1ad9b41b3605bd/macports-ports (master):

pdsh: Update to version 2.33

  • Switch to GitHub
  • Add genders variant
  • Add new co-maintainer and GitHub handles
  • Add missing dependency on ncurses

Closes: #57765
See: #53467

comment:19 Changed 6 years ago by mf2k (Frank Schima)

Thanks for your patience everyone. I wanted to fix a few things before committing.

Note: See TracTickets for help on using tickets.