patchGnome Chemistry Utils - Patches: patch #10041, Merging two molecules doesn't work...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #10041: Merging two molecules doesn't work anymore in GChemPaint

Submitter:  Jean Bréfort <jean_brefort>
Submitted:  Mon 08 Mar 2021 12:42:36 PM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  None Privacy:  Public
Assigned to:  None Open/Closed:  Open

Sun 15 May 2022 11:45:40 AM UTC, comment #3: 

It did, thank you.

Julian Sikorski <belegdol>
Group Member
Fri 13 May 2022 06:57:18 PM UTC, comment #2: 

May be intializing m_MergeButton to NULL in the constructor would fix the crash. Please try.

Jean Bréfort <jean_brefort>
Group administrator
Fri 13 May 2022 03:47:35 PM UTC, comment #1: 

On Fedora 36 gchempaint is consistently crashing on startup with this patch:

Thread 1 "gchempaint" received signal SIGBUS, Bus error.
0x00007ffff0cb03d1 in gcpSelectionTool::Activate (this=0x555555907a80) at /usr/src/debug/gnome-chemistry-utils-0.14.17-34.fc36.x86_64/plugins/paint/selection/selectiontool.cc:241
241 if (GTK_IS_WIDGET (m_MergeBtn))
Missing separate debuginfos, use: dnf debuginfo-install PackageKit-gtk3-module-1.2.5-1.fc36.x86_64 libcanberra-0.30-28.fc36.x86_64 libcanberra-gtk3-0.30-28.fc36.x86_64 libtdb-1.4.6-1.fc36.x86_64 libvorbis-1.3.7-5.fc36.x86_64
(gdb) bt
#0  0x00007ffff0cb03d1 in gcpSelectionTool::Activate() (this=0x555555907a80)
    at /usr/src/debug/gnome-chemistry-utils-0.14.17-34.fc36.x86_64/plugins/paint/selection/selectiontool.cc:241
#1  0x00007ffff7f83112 in gcp::Tool::Activate(bool) (this=0x555555907a80, bState=<optimized out>)
    at /usr/src/debug/gnome-chemistry-utils-0.14.17-34.fc36.x86_64/libs/gcp/tool.cc:144
#2  0x00007ffff7f2e727 in gcp::Application::ActivateTool(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, bool) (this=0x5555556c8650, toolname="Select", activate=<optimized out>)
    at /usr/src/debug/gnome-chemistry-utils-0.14.17-34.fc36.x86_64/libs/gcp/application.cc:531
#3  0x00007ffff0cb5526 in gcpSelectionPlugin::Populate(gcp::Application*) (this=<optimized out>, App=0x5555556c8650)
    at /usr/src/debug/gnome-chemistry-utils-0.14.17-34.fc36.x86_64/plugins/paint/selection/plugin.cc:133
#4  0x00007ffff7f2a612 in gcp::Application::Application(gcugtk::CmdContextGtk*)
    (this=0x5555556c8650, __vtt_parm=0x55555555a8b0 <VTT for gcpStandaloneApp+8>, cc=<optimized out>, this=<optimized out>, __vtt_parm=<optimized out>, cc=<optimized out>) at /usr/src/debug/gnome-chemistry-utils-0.14.17-34.fc36.x86_64/libs/gcp/application.cc:430
#5  0x000055555555779b in gcpStandaloneApp::gcpStandaloneApp() (this=0x5555556c8650, this=<optimized out>)
    at /usr/src/debug/gnome-chemistry-utils-0.14.17-34.fc36.x86_64/programs/paint/standaloneapp.cc:31
#6  main(int, char**) (argc=<optimized out>, argv=<optimized out>)
    at /usr/src/debug/gnome-chemistry-utils-0.14.17-34.fc36.x86_64/programs/paint/main.cc:79

Julian Sikorski <belegdol>
Group Member
Mon 08 Mar 2021 12:42:36 PM UTC, original submission:  

Please apply attached patch (not so clean though) to fix the issue.

Jean Bréfort <jean_brefort>
Group administrator

 

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

Attached Files
file #53199:  MergeButton0.patch added by jean_brefort (2KiB - text/x-patch - Updated patch)
file #51027:  MergeButton.patch added by jean_brefort (2KiB - text/x-patch - The patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by belegdol (Posted a comment)
  • -email is unavailable- added by jean_brefort (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.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-05-15 jean_brefort Attached File- Added MergeButton0.patch, #53199
    2021-03-08 jean_brefort Attached File- Added MergeButton.patch, #51027

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code