bugGNU Octave - Bugs: bug #44968, Ctrl-C doesn't break in GUI on a...

 
 

bug #44968: Ctrl-C doesn't break in GUI on a fresh install

Submitted by:  Andreas Weber <andy1978>
Submitted on:  Thu 30 Apr 2015 11:42:50 AM UTC  
 
Category:  GUI Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  Closed
Release:  dev Operating System:  GNU/Linux

Add a New Comment (Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 01 May 2015 03:18:37 PM UTC, comment #5: 

Fix verified.  jwe sent a note out on the Octave-Maintainers list which you might have missed, but all bug fixes for the 4.0.0 release are now happening on the stable branch.  The development branch is back to being for more experimental changes.  Accordingly, I grafted your cset on to stable here (http://hg.savannah.gnu.org/hgweb/octave/rev/f0a8c1bc3b60).

Rik <rik5>
Project Administrator
Fri 01 May 2015 09:52:28 AM UTC, comment #4: 

It is not necessary to put all default shortcuts into the default settings file. The issue is that the check for the copy shortcut in the terminal is performed from the settings file only without considering the default shortcut in case the shortcut is not found in the settings file. (The general concept is to have the default values of a preference as fallback in the read-setting-command in case the settings file is incomplete).

I have pushed changeset
http://hg.savannah.gnu.org/hgweb/octave/rev/f5e65f806123
which should fix the issue.

Torsten Lilge <ttl>
Project Member
Thu 30 Apr 2015 02:01:02 PM UTC, comment #3: 

Confirmed.  I didn't use 'LC=all' and I still had a problem.  I thought this might have something to do with the first invocation of Octave so I repeatedly shut down Octave and re-started it but the problem persisted.  Looking at the file size of qt-settings it is only 22K after a fresh trip through the install wizard.  My regular qt-settings files is ~34K.

So, I copied the '[shortcuts]' section from the working qt-settings file into the newly generated file and then sleep works.

This is what happens in the GUI when you enter the Preferences window and hit OK.  All of the settings in the various Preferences tabs are saved.

It looks like the fix should be easy.  When creating the qt-settings file for the first time the file needs to have defaults for all sections including the shortcuts section.

I've added Torsten to the CC list for this bug since he will know how to solve that quickly.

Rik <rik5>
Project Administrator
Thu 30 Apr 2015 02:00:51 PM UTC, comment #2: 

I confirm it works after deleting file qt-settings and having it recreated again by checking Preferences. This fixes all the issues I mentioned in bug #44912.

I compared the two qt-settings files - I guess that the initial one was created with an older version of Octave: for example, it does not contain codeCompletion* entries and [shortcuts] section.

Guillaume <gyom>
Thu 30 Apr 2015 11:50:03 AM UTC, comment #1: 

I can reproduce this with 3797df921988.

With --no-gui option Ctrl-C doesn't work at all.
With octave-cli, Ctrl-C works without problems.

Stefan Mahr <dac922>
Thu 30 Apr 2015 11:42:50 AM UTC, original submission:  

hg id 4ff975c58584

tl;dr: without qt-settings (fresh install), Ctrl+C doesn't break code execution. If you open the settings dialog and press "OK" it works fine.

rm -r ~/.config/octave/qt-settings
LC_ALL=C ./run-octave -f

Now click Next, Next, Finish and enter into the command window "sleep (5)" and try to break with CTRL+C (it doesn't work here).
Now open Edit, Preferences and click "OK" and try again. Now it works.

Andreas Weber <andy1978>
Project Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by ttl (Posted a comment)
  • -email is unavailable- added by gyom (Posted a comment)
  • -email is unavailable- added by rik5 (Updated the item)
  • -email is unavailable- added by rik5
  • -email is unavailable- added by dac922 (Posted a comment)
  • -email is unavailable- added by andy1978 (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

     

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-05-01 rik5 StatusReady For Test => Fixed
        Open/ClosedOpen => Closed
    2015-05-01 ttl StatusConfirmed => Ready For Test
    2015-04-30 rik5 CategoryNone => GUI
        StatusNone => Confirmed
        Carbon-Copy- => Added ttl

    Back to the top


    Powered by Savane 3.5