Re: UPDATE: devel/intellij to 12.1.2

2013-05-09 Thread Vadim Zhukov
2013/5/10 Daniel Dickman > > > >> Since you are patching bin/intellij.sh anyway, why not just rip out all > >> the JAVA_HOME/JDK-HOME goo and use javaPathHelper like all the other > Java > >> ports do? > > > > Done; files/idea wrapper is not needed anymore then, so I zapped it. This > > required

Re: UPDATE: devel/intellij to 12.1.2

2013-05-09 Thread Daniel Dickman
> >> Since you are patching bin/intellij.sh anyway, why not just rip out all >> the JAVA_HOME/JDK-HOME goo and use javaPathHelper like all the other Java >> ports do? > > Done; files/idea wrapper is not needed anymore then, so I zapped it. This > required adding a patch for readlink call, though. T

Re: UPDATE: devel/intellij to 12.1.2

2013-05-08 Thread Vadim Zhukov
> SUBST_CMD has -m [mode] now, so the extra chmod's can be avoided. Thanks for reminding, done. > Since you are patching bin/intellij.sh anyway, why not just rip out all > the JAVA_HOME/JDK-HOME goo and use javaPathHelper like all the other Java > ports do? Done; files/idea wrapper is not needed

Re: UPDATE: devel/intellij to 12.1.2

2013-05-08 Thread Stuart Henderson
SUBST_CMD has -m [mode] now, so the extra chmod's can be avoided. Vadim Zhukov wrote: >This updates Intellij IDEA to a recent version, which support >Subversion 1.7 and JDK 1.7+. > >Unfortunately, it seems to be broken on JDK 1.6, but works on 1.7+. > >I also removed some '@' in Makefile (after

UPDATE: devel/intellij to 12.1.2

2013-05-08 Thread Vadim Zhukov
This updates Intellij IDEA to a recent version, which support Subversion 1.7 and JDK 1.7+. Unfortunately, it seems to be broken on JDK 1.6, but works on 1.7+. I also removed some '@' in Makefile (after recent discussion), leaving only ${SUBST_CMD} commands hidden, and changed two invocations of t