Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 343700

Summary:

Exceptions during Build Selected Files operation are not displayed in the Build Console

Product:

[Tools] CDT

Reporter:

Alex Freidin <freidin.alex>

Component:

cdt-build-managed

Assignee:

Andrew Gvozdev <angvoz.dev>

Status:

RESOLVED FIXED

QA Contact:

Chris Recoskie <recoskie>

Severity:

normal

Priority:

CC:

cdtdoug

Version:

7.0.1

Target Milestone:

7.0.3

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
Patch	angvoz.dev: iplog+

Description Alex Freidin

2011-04-24 07:43:24 EDT

Build Identifier: M20100909-0800

If there is an exception during Build Selected Files operation, the Internal Builder doesn't print the error message to the Build Console and doesn't close the console streams. The exception handling in GeneratedMakefileBuilder.invokeInternalBuilder(IResource[] resourcesToBuild, IConfiguration...) is incomplete.

Reproducible: Always

Comment 1 Alex Freidin

2011-04-24 07:49:03 EDT

Created attachment 193965 [details]
Patch

Added the missing exception handling from the sibling method: invokeInternalBuilder(IConfiguration cfg, ...)
Patch is based on CDT 7.0.1 sources, but relevant also for HEAD (1.82 version of the file).

Comment 2 Andrew Gvozdev

2011-04-26 18:36:49 EDT

Committed on HEAD and 7.0.X with some beautification stuff if I can call it that. Thanks for the patch.

Comment 3 CDT Genie

2011-04-26 19:23:01 EDT

*** cdt cvs genie on behalf of agvozdev ***
bug 343700: Exceptions during Build Selected Files operation are not displayed in the Build Console
Patch from Alex Freidin

[*] GeneratedMakefileBuilder.java 1.83 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.managedbuilder.core/src/org/eclipse/cdt/managedbuilder/internal/core/GeneratedMakefileBuilder.java?root=Tools_Project&r1=1.82&r2=1.83

[*] GeneratedMakefileBuilder.java 1.78.2.4 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.managedbuilder.core/src/org/eclipse/cdt/managedbuilder/internal/core/GeneratedMakefileBuilder.java?root=Tools_Project&r1=1.78.2.3&r2=1.78.2.4

Comment 4 Alex Freidin

2011-04-27 03:31:12 EDT

Thanks for applying!