bugGNU Octave - Bugs: bug #53536, Ctrl+C crashes Octave and the...

 
 

bug #53536: Ctrl+C crashes Octave and the Desktop

Submitter:  Lars Kindermann <larskindermann>
Submitted:  Sat 31 Mar 2018 07:05:39 PM UTC
   
 
Category:  Interpreter Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Regression
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Mon 09 Apr 2018 09:53:02 PM UTC, comment #12: 

This solved it for me. --force-gui does not cause a crash any more.

Lars Kindermann <larskindermann>
Mon 09 Apr 2018 09:49:17 PM UTC, comment #11: 

Concurrent editing error :) Setting the status back to Fixed.

Mike Miller <mtmiller>
Group Member
Mon 09 Apr 2018 09:36:50 PM UTC, comment #10: 

We can't remove it immediately as programmers may be depending on it.  It is no longer mentioned in --help to discourage its use.  It is currently deprecated and eventually we will be able to remove it entirely.

However, the changeset that deprecated --force-gui was supposed to make it equivalent to --gui.  It doesn't seem to be equivalent for you.

I think there may be something wrong with the way we are setting up the structures for the call to getopt.

Rik <rik5>
Group administrator
Mon 09 Apr 2018 09:24:22 PM UTC, comment #9: 

You're right, there is one missing place where '--force-gui' and '--gui' are not the same. Thank you for testing and finding this serious bug. I fixed this on the stable branch now

https://hg.savannah.gnu.org/hgweb/octave/rev/9bb58f9580f4

Mike Miller <mtmiller>
Group Member
Mon 09 Apr 2018 09:08:49 PM UTC, comment #8: 

Same behaviour for stable and default branch.

Just realized that the "--force-gui" option is no longer listed by   "--help". Should it be removed from the code as "--gui" works fine?

(Apologies for not consulting the hg docs about branches)

Lars Kindermann <larskindermann>
Mon 09 Apr 2018 04:36:00 PM UTC, comment #7: 

@Lars: 'hg clone' gets you the entire repository including all branches past and present.  As Dmitri suggests, you can use 'hg update stable' or 'hg update default' to switch back and forth between branches.  Use 'hg branch' to verify which branch you are currently using.

Rik <rik5>
Group administrator
Mon 09 Apr 2018 04:25:26 PM UTC, comment #6: 

After you clone do "hg update stable" and then as usual...

Dmitri.

Dmitri A. Sergatskov <dasergatskov>
Mon 09 Apr 2018 04:21:21 PM UTC, comment #5: 

I always build from the source I got via

hg clone https://www.octave.org/hg/octave

this is the default branch, right?

How do I get to the stable branch?

Lars Kindermann <larskindermann>
Mon 09 Apr 2018 03:46:08 PM UTC, comment #4: 

Interesting.  The release you tagged was dev.  Are you building from the default branch of the Mercurial repository, or are you building the stable branch which is about to be released as 4.4.0?

Rik <rik5>
Group administrator
Mon 09 Apr 2018 08:40:17 AM UTC, comment #3: 

I use a xfce desktop (xubuntu 16.04)

Starting from a shell

/home/$USER/usr/octave/run-octave -f --force-gui

-> No problem

Creating a starter in the xfce taskbar with

/home/$USER/usr/octave/run-octave -f --force-gui

-> Ctrl-C crashes the whole desktop session

Creating a starter in the xfce taskbar with

/home/$USER/usr/octave/run-octave -f --gui

-> No problem

So the problem is caused by the --force-gui option

Lars Kindermann <larskindermann>
Mon 09 Apr 2018 04:33:20 AM UTC, comment #2: 

This works for me.  I'm running Mint 18.1 which is based on Ubuntu 16.04.  The stable tip was


parent: 25168:8c38361718a9
 When Variable Editor is tabbed, activate that tab via openvar (bug #53443).
branch: stable
commit: 5588 unknown (clean)
update: (current)


for this test.

I used the code


octave -f --gui


to start the GUI and then


while (1)
  ;
end


After that loop was started I used Ctrl+C to break the loop without problem.

Rik <rik5>
Group administrator
Wed 04 Apr 2018 08:40:04 AM UTC, comment #1: 

I can confirm the crash with ctrl+c it takes me to the login window on ubuntu 16.04
just built octave :

changeset:   25109:68039183fac4
bookmark:    @
tag:         tip
parent:      25104:1fdd1b211ed4
parent:      25108:7561fb4ab972
user:        Rik <rik@octave.org>
date:        Tue Apr 03 22:21:09 2018 -0700
summary:     maint: merge stable to default.

daniel chagas <danielch>
Sat 31 Mar 2018 07:05:39 PM UTC, original submission:  

The exact behaviour of bug #49609 is back in current dev. Version 4.2.1 is not affected.

Lars Kindermann <larskindermann>

 

(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 rik5
  • -email is unavailable- added by mtmiller (Posted a comment)
  • -email is unavailable- added by dasergatskov (Posted a comment)
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by danielch (Posted a comment)
  • -email is unavailable- added by larskindermann (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-04-09 mtmiller StatusWorks For Me Fixed
        Open/ClosedOpen Closed
    2018-04-09 rik5 StatusFixed Works For Me
        Open/ClosedClosed Open
        Carbon-Copy- Added jwe
    2018-04-09 mtmiller StatusWorks For Me Fixed
        Open/ClosedOpen Closed
    2018-04-09 rik5 StatusNone Works For Me

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code