On Wed, Nov 23, 2016 at 4:11 PM, Rafael Sadowski <[email protected]>
wrote:

> Hi everybody,
>
> Giovanni Bechis asked me to submit the new netbenas diff to ports@.
>

better to put online and send a link instead of a huge inline diff.


> So here we go, below you can find the new diff with many input from
> Giovanni Bechis, Daniel Dickman and me.
>
> Tested with C++ and Java code without any font issues on amd64.
>

did you test the cvs integration at all? i had some reports for netbeans
8.1, at least, that said that cvs support was a bit problematic.


>
> To last I would like to quote Giovanni Bechis:
>
> "even if the font issues could not be fixed for everybody it's still far
> better that what we have in-tree."
>

6.9.1 is the last version with bundled ruby on rails support. but i agree
it's time to move on and do an update.


>
> Comments?  OKs?
>

Some comments in-line from a quick read.


>
> Kind regards,
>
> Rafael Sadowski
>
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/devel/netbeans/Makefile,v
> retrieving revision 1.29
> diff -u -p -u -p -r1.29 Makefile
> --- Makefile    18 Mar 2016 23:12:16 -0000      1.29
> +++ Makefile    23 Nov 2016 20:58:19 -0000
> @@ -2,27 +2,29 @@
>
>  COMMENT=               NetBeans Java IDE
>
> -V=                     6.9.1
> -DISTNAME=              netbeans-${V}
> +V=                     8.2
> +DISTNAME=              netbeans-${V}-201609300101
> +PKGNAME=               netbeans-${V}
>

don't think you need PKGNAME if you keep the date suffix below.


>  CATEGORIES=            devel
> -REVISION=              5
>
> -HOMEPAGE=              http://www.netbeans.org/
> +HOMEPAGE=              https://www.netbeans.org/
> +
> +MAINTAINER=            Daniel Dickman <[email protected]>, \
> +                       Rafael Sadowski <[email protected]>
>

You can remove me.


>
>  # CDDL and GPLv2 dual-licensed
>  #
> -# For external components see:
> -#   http://www.netbeans.info/downloads/licence/netbeans-3rd-party.txt
> +# See:
> +# https://netbeans.org/about/legal/index.html
>  PERMIT_PACKAGE_CDROM=  Yes
>
>  MASTER_SITES=          http://download.netbeans.org/n
> etbeans/${V}/final/zip/
> -EXTRACT_SUFX=
> <http://download.netbeans.org/netbeans/$%7BV%7D/final/zip/-EXTRACT_SUFX=>
>         -201007282301-ml.zip
> +EXTRACT_SUFX=          .zip
>

please change as noted above if possible


>
>  MODULES=               java
> -MODJAVA_VER=           1.6+
> +MODJAVA_VER=           1.8+
>

ok -- i guess it's the minimum version now.


>
> -RUN_DEPENDS=           java/javaPathHelper \
> -                       devel/desktop-file-utils
> +RUN_DEPENDS=           java/javaPathHelper
>
>  NO_BUILD=              Yes
>  NO_TEST=               Yes
> @@ -31,18 +33,22 @@ WRKDIST=            ${WRKDIR}/netbeans
>
>  NB=                    ${PREFIX}/netbeans
>
> -pre-configure:
> -       @${SUBST_CMD} ${WRKSRC}/platform/lib/nbexec
> -
>  do-install:
>         ${INSTALL_DATA_DIR} ${NB}
>         @tar -czf - -C ${WRKDIR}/netbeans . | tar xzf - -C ${NB}
> +       @rm -rf ${NB}/profiler ${NB}/{ide,cnd,dlight}/bin \
> +               ${NB}/platform/modules/lib/{i386,amd64,x86}
>         @find ${NB} -name "*.dll" -o -name "*.exe" -o -name "*.cmd" \
> -               -o -name "*.bat" -o -name "*.orig" | xargs rm
> -       @${SUBST_PROGRAM} ${FILESDIR}/netbeans \
> +               -o -name "*.bat" -o -name ".lastModified" \
> +               -o -name "*.orig" | xargs rm
> +       @${INSTALL_DATA_DIR} ${PREFIX}/netbeans/cnd/bin
> +       @${INSTALL_SCRIPT} ${WRKDIR}/netbeans/cnd/bin/dorun.sh
> ${NB}/cnd/bin/
> +       @${INSTALL_DATA_DIR} ${PREFIX}/netbeans/ide/bin/nativeexecution
> +       @${INSTALL_SCRIPT} 
> ${WRKDIR}/netbeans/ide/bin/nativeexecution/hostinfo.sh
> \
> +       ${NB}/ide/bin/nativeexecution/
> +       @${SUBST_CMD} -o ${BINOWN} -g ${BINGRP} -m ${BINMODE} -c \
> +               ${FILESDIR}/netbeans \
>                 ${PREFIX}/bin/netbeans
>         @${INSTALL_DATA_DIR} ${PREFIX}/share/applications
> -       @${SUBST_DATA} ${FILESDIR}/netbeans.desktop \
> -               ${PREFIX}/share/applications/netbeans.desktop
>
>  .include <bsd.port.mk>
> Index: distinfo
> ===================================================================
> RCS file: /cvs/ports/devel/netbeans/distinfo,v
> retrieving revision 1.6
> diff -u -p -u -p -r1.6 distinfo
> --- distinfo    18 Jan 2015 03:13:19 -0000      1.6
> +++ distinfo    23 Nov 2016 20:58:19 -0000
> @@ -1,2 +1,2 @@
> -SHA256 (netbeans-6.9.1-201007282301-ml.zip) =
> NxfYxhyDqgyExDQxtrN8VvJzmd2JZRqn4CRYDOlqZow=
> -SIZE (netbeans-6.9.1-201007282301-ml.zip) = 261676411
> +SHA256 (netbeans-8.2-201609300101.zip) = rZiIM0uabB8RONyy7MyM5JIUY+hx5G
> 3vTsxhdTgWCUg=
> +SIZE (netbeans-8.2-201609300101.zip) = 351110790
> Index: patches/patch-bin_netbeans
> ===================================================================
> RCS file: patches/patch-bin_netbeans
> diff -N patches/patch-bin_netbeans
> --- /dev/null   1 Jan 1970 00:00:00 -0000
> +++ patches/patch-bin_netbeans  23 Nov 2016 20:58:19 -0000
> @@ -0,0 +1,38 @@
> +$OpenBSD$
> +
> +- use public domain Korn shell
>

change comment to "remove hidden bash dependency". any other places that
depend on bash?


> +- add OpenBSD mem size calculation
> +
> +--- bin/netbeans.orig  Fri Sep 30 01:33:04 2016
> ++++ bin/netbeans       Wed Nov 23 21:00:01 2016
> +@@ -154,6 +154,10 @@ heap_size () {
> +         mem=`cat /proc/meminfo | grep MemTotal | tr -d
> [:space:][:alpha:]:`
> +         mem=`expr $mem / 1024`
> +         ;;
> ++    OpenBSD*)
> ++        mem=`/sbin/sysctl hw.usermem | tr -d [:space:][:alpha:]=.:`
> ++        ((mem=$mem/1048576))
> ++        ;;
>

why not "sysctl -n hw.usermem"?


> +     SunOS*)
> +         mem=`/usr/sbin/prtconf | grep Memory | /usr/bin/tr -dc '[0-9]'`
> +         ;;
> +@@ -195,9 +199,9 @@ launchNbexec() {
> +     done | head -n 1`
> +     sh=sh
> +     # #73162: Ubuntu uses the ancient Bourne shell, which does not
> implement trap well.
> +-    if [ -x /bin/bash ]
> ++    if [ -x /bin/ksh ]
> +     then
> +-        sh=/bin/bash
> ++        sh=/bin/ksh
> +     fi
>

just remove this whole "if" block and the comment about bug 73162. Same
thing needed in harness/launchers/app.sh"?


> +     if [ "${founduserdir}" = "yes" ]; then
> +         exec $sh "$nbexec" "$@"
> +@@ -230,6 +234,7 @@ case "`uname`" in
> +             --jdkhome '"$netbeans_jdkhome"' \
> +             --branding nb \
> +             --clusters '"$netbeans_clusters"' \
> ++            -J-Dawt.useSystemAAFontSettings=lcd \
> +             -J-Dnetbeans.importclass=org.netbeans.upgrade.AutoUpgrade \
> +             
> -J-Dnetbeans.accept_license_class=org.netbeans.license.AcceptLicense
> \
> +             ${netbeans_default_options} \
> Index: patches/patch-etc_netbeans_conf
> ===================================================================
> RCS file: /cvs/ports/devel/netbeans/patches/patch-etc_netbeans_conf,v
> retrieving revision 1.3
> diff -u -p -u -p -r1.3 patch-etc_netbeans_conf
> --- patches/patch-etc_netbeans_conf     12 Nov 2010 12:26:16 -0000
> 1.3
> +++ patches/patch-etc_netbeans_conf     23 Nov 2016 20:58:19 -0000
> @@ -3,14 +3,14 @@ $OpenBSD: patch-etc_netbeans_conf,v 1.3
>  Turn autoupdate off by default. The relevant options are documented here:
>         http://wiki.netbeans.org/FaqPluginManagerCustomization
>
> ---- etc/netbeans.conf.orig     Thu Jul 29 02:06:10 2010
> -+++ etc/netbeans.conf  Tue Nov  9 21:48:58 2010
> -@@ -3,7 +3,7 @@ netbeans_default_userdir="${HOME}/.netbeans/6.9"
> +--- etc/netbeans.conf.orig     Fri Sep 30 01:33:04 2016
> ++++ etc/netbeans.conf  Wed Nov 23 21:17:08 2016
> +@@ -43,7 +43,7 @@ netbeans_default_cachedir="${DEFAULT_CACHEDIR_ROOT}/8.
> + # Concurrent Mark & Sweep garbage collector.
> + # (see http://wiki.netbeans.org/FaqGCPauses)
> + #
> +-netbeans_default_options="-J-client -J-Xss2m -J-Xms32m
> -J-Dapple.laf.useScreenMenuBar=true -J-Dapple.awt.graphics.UseQuartz=true
> -J-Dsun.java2d.noddraw=true -J-Dsun.java2d.dpiaware=true
> -J-Dsun.zip.disableMemoryMapping=true"
> ++netbeans_default_options="-J-client -J-Xss2m -J-Xms32m
> -J-Dapple.laf.useScreenMenuBar=true -J-Dapple.awt.graphics.UseQuartz=true
> -J-Dsun.java2d.noddraw=true -J-Dsun.java2d.dpiaware=true
> -J-Dsun.zip.disableMemoryMapping=true -J-Dplugin.manager.check.updates=false
> -J-Dplugin.manager.check.interval=NEVER"
>
> - # Options used by NetBeans launcher by default, can be overridden by
> explicit
> - # command line switches:
> --netbeans_default_options="-J-client -J-Xss2m -J-Xms32m
> -J-XX:PermSize=32m -J-XX:MaxPermSize=200m -J-Dapple.laf.useScreenMenuBar=true
> -J-Dapple.awt.graphics.UseQuartz=true -J-Dsun.java2d.noddraw=true"
> -+netbeans_default_options="-J-client -J-Xss2m -J-Xms32m
> -J-XX:PermSize=32m -J-XX:MaxPermSize=200m -J-Dapple.laf.useScreenMenuBar=true
> -J-Dapple.awt.graphics.UseQuartz=true -J-Dsun.java2d.noddraw=true
> -J-Dplugin.manager.check.updates=false -J-Dplugin.manager.check.inter
> val=NEVER"
> - # Note that a default -Xmx is selected for you automatically.
> - # You can find this value in var/log/messages.log file in your userdir.
> - # The automatically selected value can be overridden by specifying
> -J-Xmx here
> + # Default location of JDK:
> + # (set by installer or commented out if launcher should decide)
> Index: patches/patch-platform_lib_nbexec
> ===================================================================
> RCS file: /cvs/ports/devel/netbeans/patches/patch-platform_lib_nbexec,v
> retrieving revision 1.1
> diff -u -p -u -p -r1.1 patch-platform_lib_nbexec
> --- patches/patch-platform_lib_nbexec   9 Jul 2010 07:35:04 -0000
>  1.1
> +++ patches/patch-platform_lib_nbexec   23 Nov 2016 20:58:19 -0000
> @@ -1,46 +0,0 @@
> -$OpenBSD: patch-platform_lib_nbexec,v 1.1 2010/07/09 07:35:04 landry Exp $
> ---- platform/lib/nbexec.orig   Thu Jun 10 19:59:48 2010
> -+++ platform/lib/nbexec        Sun Jun 20 02:16:30 2010
> -@@ -357,7 +357,7 @@ detect_system_proxy () {
> - }
> -
> - detect_gnome_proxy () {
> --    gconftool=/usr/bin/gconftool-2
> -+    gconftool=${LOCALBASE}/bin/gconftool-2
> -     if [ -x  $gconftool ] ; then
> -         proxy_mode=`$gconftool --get /system/proxy/mode 2>/dev/null`
> -         if [ "$proxy_mode" = "manual" ] ; then
> -@@ -366,7 +366,7 @@ detect_gnome_proxy () {
> -             http_proxy_tmp=$http_proxy_host:$http_proxy_port
> -             http_non_proxy_hosts=`$gconftool --get
> /system/http_proxy/ignore_hosts 2>/dev/null`
> -             if [ $? ] ; then
> --                http_non_proxy_hosts=`echo $http_non_proxy_hosts |
> /bin/sed 's/\]//'`
> -+                http_non_proxy_hosts=`echo $http_non_proxy_hosts |
> /usr/bin/sed 's/\]//'`
> -             fi
> -             socks_proxy_host=`$gconftool --get /system/proxy/socks_host
> 2>/dev/null`
> -             socks_proxy_port=`$gconftool --get /system/proxy/socks_port
> 2>/dev/null`
> -@@ -389,18 +389,18 @@ detect_gnome_proxy () {
> - detect_kde_proxy () {
> -     kioslaverc="${HOME}/.kde/share/config/kioslaverc"
> -     if [ -f $kioslaverc ] ; then
> --        if /bin/grep 'ProxyType=1' "$kioslaverc" >/dev/null 2>&1; then
> --            http_proxy_tmp=`/bin/grep 'httpProxy=http://' "$kioslaverc"`
> -+        if /usr/bin/grep 'ProxyType=1' "$kioslaverc" >/dev/null 2>&1;
> then
> -+            http_proxy_tmp=`/usr/bin/grep 'httpProxy=http://'
> "$kioslaverc"`
> -             if [ $? ] ; then
> --                http_proxy_tmp=`echo $http_proxy_tmp | /bin/sed
> 's/httpProxy=http:\/\///'`
> -+                http_proxy_tmp=`echo $http_proxy_tmp | /usr/bin/sed
> 's/httpProxy=http:\/\///'`
> -                 return 0
> -             fi
> --            http_non_proxy_hosts=`/bin/grep 'NoProxyFor=' "$kioslaverc"`
> -+            http_non_proxy_hosts=`/usr/bin/grep 'NoProxyFor='
> "$kioslaverc"`
> -             if [ $? ] ; then
> --                http_non_proxy_hosts=`echo $http_non_proxy_hosts |
> /bin/sed 's/NoProxyFor=//'`
> -+                http_non_proxy_hosts=`echo $http_non_proxy_hosts |
> /usr/bin/sed 's/NoProxyFor=//'`
> -             fi
> -         else
> --            if /bin/grep 'ProxyType=0' "$kioslaverc" >/dev/null 2>&1;
> then
> -+            if /usr/bin/grep 'ProxyType=0' "$kioslaverc" >/dev/null
> 2>&1; then
> -                 detect_system_proxy
> -                 if [ -z "$http_proxy_tmp" ]; then
> -                     http_proxy_tmp="DIRECT"
> -share/applications/netbeans.desktop
>  share/doc/pkg-readmes/${FULLPKGNAME}
> -@exec %D/bin/update-desktop-database
> -@unexec-delete %D/bin/update-desktop-database
> Index: pkg/README
> ===================================================================
> RCS file: /cvs/ports/devel/netbeans/pkg/README,v
> retrieving revision 1.2
> diff -u -p -u -p -r1.2 README
> --- pkg/README  2 Jun 2011 13:41:38 -0000       1.2
> +++ pkg/README  23 Nov 2016 20:58:19 -0000
> @@ -4,21 +4,6 @@ $OpenBSD: README,v 1.2 2011/06/02 13:41:
>  | Running ${FULLPKGNAME} on OpenBSD
>  +-----------------------------------------------------------------------
>
> -Help System
> -===========
> -
> -NetBeans relies on the presence of a firefox binary for access to
> -the online help system. Firefox is not set up as a runtime dependency
> -as this dependency is optional.
> -
> -If you have Firefox 3.5 on your system you may want to set up the
> -following symbolic link:
> -
> -ln -s ${LOCALBASE}/bin/firefox35 ${LOCALBASE}/bin/firefox
> -
> -You can also customize this setting within the IDE by going to
> -Tools> Options> General> Web Browser.
> -
>  Font issues in the output window
>  ================================
>
>
>
>
>

Reply via email to