IzPack

Error message if configuration file not readable can be misleading

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 3.11.0
  • Fix Version/s: 4.0.0
  • Component/s: Compiler
  • Labels:
    None
  • Environment:
    Slackware Linux 12.0, Sun Java 1.6.0_04
  • Patch Submitted:
    Yes
  • Number of attachments :
    1

Description

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.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Ansgar Konermann added a comment -

Affects 3.11.0, Compiler

Show
Ansgar Konermann added a comment - Affects 3.11.0, Compiler
Hide
Permalink
Ansgar Konermann added a comment -

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.

Show
Ansgar Konermann added a comment - 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.
Hide
Permalink
Ansgar Konermann added a comment -

Priority should probably be lowered. Unfortunately, I'm unable to edit this issue anymore.

Show
Ansgar Konermann added a comment - Priority should probably be lowered. Unfortunately, I'm unable to edit this issue anymore.
Hide
Permalink
Julien Ponge added a comment -

Thanks for the report, I will handle it in the next few days.

Show
Julien Ponge added a comment - Thanks for the report, I will handle it in the next few days.
Hide
Permalink
Julien Ponge added a comment -

Thanks for the patch, it has been applied with minor changes.

Show
Julien Ponge added a comment - Thanks for the patch, it has been applied with minor changes.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:

Time Tracking

Estimated:
2h
Original Estimate - 2 hours
Remaining:
2h
Remaining Estimate - 2 hours
Logged:
Not Specified
Time Spent - Not Specified
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.