History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: IZPACK-35
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Julien Ponge
Reporter: Ansgar Konermann
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
IzPack

Error message if configuration file not readable can be misleading

Created: 21/Mar/08 05:45 PM   Updated: 22/Mar/08 01:03 PM
Component/s: Compiler
Affects Version/s: 3.11.0
Fix Version/s: 4.0.0

Time Tracking:
Original Estimate: 2 hours
Original Estimate - 2 hours
Remaining Estimate: 2 hours
Remaining Estimate - 2 hours
Time Spent: Not Specified
Remaining Estimate - 2 hours

File Attachments: 1. File CompilerConfig.diff (2 kb)

Environment: Slackware Linux 12.0, Sun Java 1.6.0_04

Patch Submitted: Yes


 Description  « Hide
In CompilerConfiguration.java (line 1735), the code checks whether the configuration file is readable. The error message specified here is somewhat misleading sometimes. It just says "invalid file". If you're new to IzPack and don't know the reason for this message, "invalid file" could also be interpreted as "contains invalid content", meaning "syntactically wrong".

It would be nice if the error messages were mere distinguished.



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Ansgar Konermann - 21/Mar/08 05:46 PM
Affects 3.11.0, Compiler

Ansgar Konermann - 21/Mar/08 05:53 PM
Patch should give an idea how this could be improved.

However, it needs to be quality-checked: I was unable to build complete project using ant due to time constraints.


Ansgar Konermann - 21/Mar/08 05:56 PM
Priority should probably be lowered. Unfortunately, I'm unable to edit this issue anymore.

Julien Ponge - 22/Mar/08 05:52 AM
Thanks for the report, I will handle it in the next few days.

Julien Ponge - 22/Mar/08 01:03 PM
Thanks for the patch, it has been applied with minor changes.