Re: [macports-ports] branch master updated: pingus: new port submission

2016-12-03 Thread Lawrence Velázquez
> On Dec 3, 2016, at 6:01 AM, Ken Cunningham > wrote: > > tobypeterson pushed a commit to branch master > in repository macports-ports. > > https://github.com/macports/macports-ports/commit/d9673942b719a31aee1623f64d765e57de401dc5 > >

Re: [macports-ports] branch master updated: pingus: new port submission

2016-12-03 Thread Brandon Allbery
On Sat, Dec 3, 2016 at 3:20 PM, Lawrence Velázquez wrote: > +reinplace "s|\"$@\"||g" > ${destroot}${applications_dir}/Pingus.app/Contents/MacOS/pingus+} > > This is…strange. What exactly are you replacing here? > Likely making it not pass the (automatically generated and passed) session ID

Re: [macports-ports] branch master updated: pingus: new port submission

2016-12-03 Thread Ken Cunningham
Hi Larry, Thanks for the feedback. Apparently Pingus is very popular, and not on homebrew! A scoop! Hope people enjoy it (or at least their kids do). > This should be "GPL-3+”. Ah. I saw this

Re: [macports-ports] branch master updated: pingus: new port submission

2016-12-03 Thread Brandon Allbery
On Sat, Dec 3, 2016 at 3:38 PM, Ken Cunningham < ken.cunningham.web...@gmail.com> wrote: > Yeah - that was interesting. The bin/pingus launch script had this: > > #!/bin/sh > exec "/Applications/MacPorts/Pingus.app/Contents/MacOS/libexec/pingus" > --datadir "/Applications/MacPorts/Pingus.app/Conte

Re: [macports-ports] branch master updated: pingus: new port submission

2016-12-03 Thread Brandon Allbery
On Sat, Dec 3, 2016 at 3:43 PM, Brandon Allbery wrote: > > Get rid of any parameter that starts with the characters "-psn_". This is > added by the bundle launcher, expected by native applications, and will > confuse most Linux-originated programs. I should note that doing this properly is much