not really sure who's to blame here - pls feel free to reassign

According to guidelines[1], any regression is an P1.

[1]http://wiki.netbeans.org/BugPriorityGuidelines

This is minor issue, not regression in functionality. Does this issue prevent you to do something?

No, in this point you are right, this is just annoying since I use really huge Output views...

Regards

Could you just say me which project it is? I will try to create a patch if possible.

Regards

core-main #ca3d1b5648ab

Integrated into 'main-golden', will be available in build *200904170201* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/ca3d1b5648ab
User: Tomas Holy <t_h@netbeans.org>
Log: #162159: Output window does not respect 'Always Show Output' anymore

v. 200904220201

I am still seeing this bug in NetBeans IDE 6.8 (Build 200912041610) on both OS X and Windows Vista.

Windows Vista Java: 1.6.0_18; Java HotSpot(TM) Client VM 16.0-b13
OS X Java: Latest Mac update (1.6.0_17, I believe)

I am also finding this to be very annoying

Could you send us steps to reproduce? I can help to try to fix this.

I don't have to do anything special. I just created a new Java application, made sure the Always Show Output box is unchecked, hit F11 to build the application, and the output window opens.

Yes, you are right, it is back :(
I will try to take a look at this next week.

Thanks.

Take a look at line 463 in TargetExecutor.java

The defined interestingOutputCallback is being passed into the ant executor, which ends up calling it through the logging framework. I think code needs to be added to the ant event logging to determine if the given event is "interesting" or not.

Yes, I saw that. It is to be called when an input field is available. I have seen Jesse, the change author, is already looking this. Hopefully we will have an fix soon :)

Well, the problem was already identified by dkkopp and confirmed by me but should be defined yet how to fix it. Maybe some NetFIXer [1] could take a look at this?

[1] http://wiki.netbeans.org/NetFIX

Yes, Jesse agreed to review & integrate possible bug fix from NetFIX team.

Hi, I'd like to submit a patch but I'm not sure about it because I don't know the whole NetBeans API, so I am afraid about collateral damages.

For that reason, I'd like some community help.

On the Ant Module [1] within org.apache.tools.ant.module.bridge.impl.NbBuildLogger class at #768 line - InputOutput getIO() method.

before:

    public InputOutput getIO() {
        interestingOutputCallback.run();
        return io;
    }

after:

    public InputOutput getIO() {

        if (AntSettings.getAlwaysShowOutput()) {
            interestingOutputCallback.run();
        }

        return io;
    }

[1] ~/main-silver/o.apache.tools.ant.module

Regression from bug #167386. Also t_h's original fix was not correct since it was intentional that Always Show Output be disabled by default (see bug #87801 for background).

Fix is not perfect since it introduces a minor regression: a hyperlink (StandardLogger.PartiallyLinkedLine.println) will no longer show the output window. Not solvable without an API extension to AntSession to show output, since getIO no longer does so, and println in this case is called with a null listener.

core-main #66582d893086

Integrated into 'main-golden', will be available in build *201004020200* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/66582d893086
User: Jesse Glick <jglick@netbeans.org>
Log: #162159: [67cat] Output window does not respect 'Show Always Output' anymore

OK, I have removed this issue from NetFIX Pool. Thanks for your patch attempt Roberto anyway!

Well, it is working again at least for me :)
Thanks everyone for the fix.
v. 100405-ba96ec22818f