This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Originally 6917708: When running the /.../100101/inst/intel-S2/ build on opensolaris b130/x86 I noticed the below messages in the command line suggesting Sun grep is called instead of gnu grep somewhere. I am not aware what action trigerred these messages though. I was running the GUI, browsing various C++ sources. $ sunstudio /bin/grep: illegal option -- q Usage: grep -hblcnsviw pattern file . . . /bin/grep: illegal option -- q Usage: grep -hblcnsviw pattern file . . . *** (#1 of 1): 2010-01-18 05:03:04 PST pavel heimlich
No clear who is running this command. Making it a P2 until we know and understand what feature may be affected.
It may come from Netbean's startup code nbexec: d-spblab-32-168[567] pwd /Users/thp/netbeans-dev/cnd-main/nbbuild/netbeans/platform11/lib d-spblab-32-168[568] grep "grep -q" * nbexec: if /bin/grep -q 'ProxyType=1' "$kioslaverc" ; then nbexec: if /bin/grep -q 'ProxyType=0' "$kioslaverc" ; then nbexec: if /usr/bin/grep -q "ProxyAuto.*: *1" ${scutil_out} >/dev/null ; then nbexec: if /usr/bin/grep -q "HTTPEnable *: *1" ${scutil_out} ; then
Reassigning to platform for further evaluation.
platform/lib/nbexec contains -q. Please suggest replacement.
Created attachment 93783 [details] platform/lib/boot.jar
A suggestion for the portable shell scripts is described here: http://www.gnu.org/software/grep/manual/grep.html#General-Output-Control "... Portability note: unlike gnu grep, 7th Edition Unix grep did not conform to posix, because it lacked ‘-q’ and its ‘-s’ option behaved like gnu grep's ‘-q’ option. ... Portable shell scripts should avoid both ‘-q’ and ‘-s’ and should redirect standard and error output to /dev/null instead. (‘-s’ is specified by posix.) "
OK, thanks for the hint. I tried: core-main#177373b33a05. Works for me on Linux, Thomas shall however test it on Solaris.
Integrated into 'main-golden', will be available in build *201002040200* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress) Changeset: http://hg.netbeans.org/main/rev/177373b33a05 User: Jaroslav Tulach <jtulach@netbeans.org> Log: #179662: Sending output to /dev/null, instead of using grep -q
The code change looks good, but not sure I know how to test it. I didn't report the problem and I have never noticed it myself. The report came from Sun's internal bug report system as CR 6917708.
Victor, could you please check on your OpenSolaris system? Thanks.
Dmitry, I've tested behavior of the statements that are used in the script on OpenSolaris. As expected the redirecting of output to the /dev/null instead of the '-q' option is working well for a non-GNU version of the grep command. But, unfortunately I can't help with full testing of the use cases. There are at least two use cases (i.e. environments) where the fixed statements will be used: 1. OpenSolaris + KDE 2. Mac OS I have not such environments installed. Full testing should also rely on launch of the script from sunstudio, but not netbeans.
Hi, if the fix consists in applying the changeset to /opt/sunstudio12.1/netbeans/platform9/lib/nbexec and /opt/sunstudio12.1/prod/nb-dbxtool/platform9/lib/nbexec then I can do that for KDE/opensolaris
Please also provide a patch for this fix in 6.8. Sun Studio is based on 6.8 and mainly used on Solaris.
Could QE verify this issue in the trunk in order that it can be ported into the release68_fixes branch?
Notes related to the patch: #1 Looks good and passed my eye-review #2 Due to the nature of the launcher for release68_fixes repository it *must* be accompanied by a patch to o.n.bootstrap/nbproject/project.properties to add a line "nbm.is.global=true" to enforce installation in <installdir> instead of <userdir>. IDE launcher does not look for Platform launcher in <userdir>. #3 It's explicitly *forbidden* to set hard dependency on this new version of o.n.bootstrap module as it could cause cascaded unsatisfied dependency issues when it would not be possible to install this update due to insufficient permissions to modify <installdir>'s filesystem (so called root-user scenario). #4 Little nitpicking - choose either /bin/grep or /usr/bin/grep and use just that one
The whiteboard 68patch-candidate has been removed based on the internal discussion. The issue is fixed in the trunk and it will be part of upcoming IDE release. There are no plans to port the fix into the NetBeans IDE 6.8.