symbol update reads the symbol from the cache

Bug #698504 reported by werner2101
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gEDA
Fix Released
Low
Unassigned

Bug Description

If I update a symbol that I've used before, then the update fails.

To reproduce:
* place a (user defined) symbol
* goto the symbol (hierarchy symbol)
* edit the symbol and save it
* go the hierarchy up
* update the symbol --> nothing happens.

If you refresh the library in the component chooser before updating, the update works fine.

geda version: 9b075276bf13f694b9d7f2cf64197da03b7d1cc6

Regards
Werner

Revision history for this message
Peter Clifton (pcjc2) wrote :

Assigning Peter B, since he wrote the caching code, therefore deserves the "blame" ;)

Revision history for this message
Peter Clifton (pcjc2) wrote :

Fixed (worked around) by invalidating the symbol cache before fetching the updated symbol data from the library.

NB: No API or ABI changes.

A more complete fix will be to add API for invalidating a specific symbol by name.

Peter TB Brett (peter-b)
Changed in geda:
assignee: nobody → Peter TB Brett (peter-b)
status: New → Confirmed
Peter TB Brett (peter-b)
tags: added: libgeda
Revision history for this message
gpleda.org commit robot (gpleda-launchpad-robot) wrote :

Bug was fixed by a commit
git master commit ebfe6ae6a76b6a44a0bbe2d184bf99bfee7d97bf
http://git.gpleda.org/?p=gaf.git;a=commit;h=ebfe6ae6a76b6a44a0bbe2d184bf99bfee7d97bf

commit ebfe6ae6a76b6a44a0bbe2d184bf99bfee7d97bf
Author: Peter TB Brett <email address hidden>
Commit: Peter TB Brett <email address hidden>

    Add s_clib_symbol_invalidate_data().

    This allows a symbol's data to be reloaded from disk without
    invalidating the full symbol data cache.

    Closes-bug: lp-698504

Changed in geda:
status: Confirmed → Fix Committed
Peter TB Brett (peter-b)
Changed in geda:
assignee: Peter TB Brett (peter-b) → nobody
milestone: none → 1.7.2
status: Fix Committed → Confirmed
status: Confirmed → Fix Committed
Peter TB Brett (peter-b)
Changed in geda:
status: Fix Committed → Fix Released
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.