Bug 93904 - Batch compiler -log option broken for some values
Summary: Batch compiler -log option broken for some values
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 minor (vote)
Target Milestone: 3.1 M7   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-05-06 09:13 EDT by Maxime Daniel CLA
Modified: 2005-05-19 06:54 EDT (History)
0 users

See Also:


Attachments
Batch test case (43.40 KB, patch)
2005-05-13 10:25 EDT, Maxime Daniel CLA
no flags Details | Diff
Batch test case (46.33 KB, patch)
2005-05-19 06:54 EDT, Maxime Daniel CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Maxime Daniel CLA 2005-05-06 09:13:26 EDT
Build: N20050506
When using the batch compiler -log option with a file name that has no 
extension at all, like my_output, the resulting file is empty, and the console 
is not fed either.
Note: behavior with my my_output.txt, my_output.xml and my_output.other_ext 
all OK (that is, log into the file, as XML for .xml files and text for others, 
and print the log as text to console).
(Trivial) workaround is to always use log file names that have an extension.
Comment 1 Olivier Thomann CLA 2005-05-10 10:19:20 EDT
Fixed and released in HEAD.
Changes in org.eclipse.jdt.internal.compiler.batch.Main.Logger.setLog(String).
No test added for the batch compiler.
Comment 2 Maxime Daniel CLA 2005-05-13 10:25:20 EDT
Created attachment 21104 [details]
Batch test case

This batch test case shows that the log option now works as expected with .xml,
.txt and no extension files.
(This patch also includes test cases related to #92398 and superceedes the
matching patches there.)
Comment 3 Maxime Daniel CLA 2005-05-13 10:25:44 EDT
Verified for 3.1 M7 using build I20050512-2035 + jdt.core HEAD.
Comment 4 Maxime Daniel CLA 2005-05-19 06:54:18 EDT
Created attachment 21415 [details]
Batch test case

Supersedes previous patch because of changes initiated for
https://bugs.eclipse.org/bugs/show_bug.cgi?id=92398.
Note that only part of it directly addresses this bug.