error messages in status window should be more explicit

Bug #698578 reported by KaiMartin
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
gEDA
Triaged
Medium
Unassigned

Bug Description

If gschem encounters a line in a config file it does not understand, the message sent to the status window lacks the keywords "error" or "warning". Subsequently, gschem quits parsing the resource file at the offending line and resumes the start-up procedure. This may result in unexpected behavior.

Suggestion:
The message might read: "gschem encountered a problem while reading <filename>, line <line number>. The rest of the file remains unread."
This message should go to stdout, too.

---<(kaimartin)>---

Peter TB Brett (peter-b)
Changed in geda:
importance: Undecided → High
status: New → Confirmed
Peter TB Brett (peter-b)
Changed in geda:
milestone: none → 1.8.0
Revision history for this message
Peter TB Brett (peter-b) wrote :

I'm currently working on enhancements to libgeda to support this.

Changed in geda:
assignee: nobody → Peter TB Brett (peter-b)
status: Confirmed → In Progress
tags: added: libgeda
Revision history for this message
KaiMartin (kmk-familieknaak) wrote :

Nice!
This will certainly improve user experience.
---<)kaimartin(>---

Revision history for this message
Peter TB Brett (peter-b) wrote :

Hi KaiMartin,

I've just committed some changes to unstable which:

- Modify all libgeda-using apps other than gschem to print errors on stderr and exit with non-zero status if they encounter errors while loading initial configuration files.
- Modify gschem to show error dialogs for any errors loading initial configuration files.

Please test them, and give me feedback. Note that there will still not be exit errors/error dialogs for per directory configuration file errors found while loading schematics, as those unfortunately use a rather different codepath which is going to be quite difficult to refactor in a "nice" way.

I'm going to move onto other issues for now; this bug should be revisited sometime in the future, at which time local configuration should be factored out of f_open_flags() and applications updated as necessary. Lowering importance to medium, since most serious aspects of bug have been fixed.

Changed in geda:
importance: High → Medium
status: In Progress → Triaged
assignee: Peter TB Brett (peter-b) → nobody
Revision history for this message
Peter TB Brett (peter-b) wrote :

It will not be possible to improve on this issue without major changes that aren't feasible to fit into 1.8.0, sorry.

Changed in geda:
milestone: 1.8.0 → none
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.