On 11/24/16 06:37, Daniel Dickman wrote:
> 
> 
> On Wed, Nov 23, 2016 at 4:11 PM, Rafael Sadowski <[email protected] 
> <mailto:[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.
> 
I tested it today for the first time and netbeans downloaded a plugin in 
~/.netbeans/8.2/modules/org-netbeans-modules-versioning-system-cvss.jar, I 
tried some simple stuff and it seems to work.
As for maintainership you can add me instead if Daniel, I am using it daily for 
php stuff.
 Cheers
  Giovanni
 
> 
> 
>     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] 
> <mailto:[email protected]>>, \
>     +                       Rafael Sadowski <[email protected] 
> <mailto:[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 
> <http://www.netbeans.info/downloads/licence/netbeans-3rd-party.txt>
>     +# See:
>     +# https://netbeans.org/about/legal/index.html 
> <https://netbeans.org/about/legal/index.html>
>      PERMIT_PACKAGE_CDROM=  Yes
> 
>      MASTER_SITES=          
> http://download.netbeans.org/netbeans/${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 <http://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+hx5G3vTsxhdTgWCUg=
>     +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 
> <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 
> <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.interval=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