[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

gEDA-bug: [Bug 698565] Re: Save button doesn't respect permissions



On Saturday 02 July 2011 15:39:41 Eivind Kvedalen wrote:
> I'm not sure if backup (and auto-save) should be part of libgeda at
> all 

I'm pretty certain they shouldn't.

> I can try to refactor the code if you think it is worth it.

I think that this part of the libgeda API needs a serious rethink, and 
possibly a redesign.  I don't think it's worth putting a lot of effort 
into refactoring the current API.

Thanks for your work on this patch -- I'm sorry for taking so long to 
get your patches reviewed & merged.

                            Peter

-- 
Peter Brett <peter@xxxxxxxxxxxxx>
Remote Sensing Research Group
Surrey Space Centre


** Changed in: geda
    Milestone: 1.8.0 => 1.7.2

-- 
You received this bug notification because you are a member of gEDA Bug
Team, which is subscribed to gEDA.
https://bugs.launchpad.net/bugs/698565

Title:
  Save button doesn't respect permissions

Status in GPL Electronic Design Automation tools:
  Fix Committed

Bug description:
  File->Save _sometimes_ fails to respect file-permissions.

  For example, when modifying a file with permissions:
  -r--r--r-- 1 pcjc2 pcjc2 510 2008-12-27 10:23 /home/pcjc2/geda/share/gEDA/sym/analog/resistor-2.sym

  The log window often gets the following text:
  o_save: Could not open [/home/pcjc2/geda/share/gEDA/sym/analog/resistor-2.sym]
  Could NOT save page [/home/pcjc2/geda/share/gEDA/sym/analog/resistor-2.sym]
  (Arguably we just need one descriptive message with a reason).

  However; on some occasions, the file gets overritten, creating a new file with permissions:
  -rw-r--r-- 1 pcjc2 pcjc2 490 2008-12-27 10:29 /home/pcjc2/geda/share/gEDA/sym/analog/resistor-2.sym

  Obviously this is not good!

To manage notifications about this bug go to:
https://bugs.launchpad.net/geda/+bug/698565/+subscriptions


_______________________________________________
geda-bug mailing list
geda-bug@xxxxxxxxxxxxxx
http://www.seul.org/cgi-bin/mailman/listinfo/geda-bug