Bug 201211 - [PATCH] x11-wm/obmenu: SIGUSR2 is hardcoded as signal #12, causing openbox to quit when it saves
Summary: [PATCH] x11-wm/obmenu: SIGUSR2 is hardcoded as signal #12, causing openbox to...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Thomas Zander
URL:
Keywords: patch, patch-ready
Depends on:
Blocks:
 
Reported: 2015-06-30 03:07 UTC by James Lu
Modified: 2015-07-17 15:57 UTC (History)
4 users (show)

See Also:
bugzilla: maintainer-feedback? (horia)


Attachments
Patch for obmenu (589 bytes, patch)
2015-06-30 03:07 UTC, James Lu
no flags Details | Diff
obmenu.diff (1.13 KB, patch)
2015-07-06 15:11 UTC, Horia Racoviceanu
horia: maintainer-approval+
Details | Diff
obmenu-1.0_8-Bug201211-Bug198031-10amd64.log (33.26 KB, text/plain)
2015-07-06 15:13 UTC, Horia Racoviceanu
no flags Details
obmenu-1.0_8-Bug201211-Bug198031-10i386.log (33.20 KB, text/plain)
2015-07-06 15:14 UTC, Horia Racoviceanu
no flags Details
obmenu-1.0_8-Bug201211-Bug198031-9amd64.log (33.12 KB, text/plain)
2015-07-06 15:14 UTC, Horia Racoviceanu
no flags Details
obmenu-1.0_8-Bug201211-Bug198031-9i386.log (32.95 KB, text/plain)
2015-07-06 15:15 UTC, Horia Racoviceanu
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description James Lu 2015-06-30 03:07:50 UTC
Created attachment 158175 [details]
Patch for obmenu

Upon saving changes, Obmenu is supposed to make Openbox reload its config by sending SIGUSR2. However, it hardcodes this signal as signal #12, which works on Linux, but not on FreeBSD (Python reports signal.SIGUSR2 to be signal number 31). Instead, whenever Obmenu saves, Openbox dumps core and exits on signal #12.

A patch to fix this is attached.
Comment 1 Horia Racoviceanu 2015-07-06 15:11:28 UTC
Created attachment 158417 [details]
obmenu.diff

Merging Bug 201211 with Bug 198031

I accept this patch, with modifications.
Thank you.

- Bump PORTREVISION
- Add USES=shebangfix for SHEBANG_FILES=obmenu obxml.py pipes/obm-xdg
- Add NO_ARCH
- Patch obxml.py to show command in execute actions [1]
- Patch obmenu: SIGUSR2 is hardcoded as signal #12,
  causing openbox to quit when it saves [2]

[1]
PR:		198031
Submitted by:	JochenF

[2]
PR:		201211
Submitted by:	James Lu

# portlint -AC
WARN: Makefile: for new port, make $FreeBSD$ tag in comment section empty, to make SVN happy.
WARN: Makefile: new ports should not set PORTREVISION.
0 fatal errors and 2 warnings found.
Comment 2 Horia Racoviceanu 2015-07-06 15:13:48 UTC
Created attachment 158418 [details]
obmenu-1.0_8-Bug201211-Bug198031-10amd64.log
Comment 3 Horia Racoviceanu 2015-07-06 15:14:16 UTC
Created attachment 158419 [details]
obmenu-1.0_8-Bug201211-Bug198031-10i386.log
Comment 4 Horia Racoviceanu 2015-07-06 15:14:43 UTC
Created attachment 158420 [details]
obmenu-1.0_8-Bug201211-Bug198031-9amd64.log
Comment 5 Horia Racoviceanu 2015-07-06 15:15:03 UTC
Created attachment 158421 [details]
obmenu-1.0_8-Bug201211-Bug198031-9i386.log
Comment 6 Kubilay Kocak freebsd_committer freebsd_triage 2015-07-06 15:15:43 UTC
Horia, can you please:

 * Set maintainer-approval for your own patch
 * Set one of the issues  you referenced in comment 1 as depending on, or blocking the other
 * Set the 'obsolete flag' on any patches in either issue that is contained in this issues latest patch.
Comment 7 commit-hook freebsd_committer freebsd_triage 2015-07-17 15:43:23 UTC
A commit references this bug:

Author: riggs
Date: Fri Jul 17 15:42:31 UTC 2015
New revision: 392381
URL: https://svnweb.freebsd.org/changeset/ports/392381

Log:
  Fix runtime error:
  hardcoded signal mapping causes openbox to quit while saving

  PR:		201211
  Submitted by:	glolol1@hotmail.com
  Reviewed by:	horia@racoviceanu.com (maintainer)
  Approved by:	horia@racoviceanu.com (maintainer)
  MFH:		2015Q3

Changes:
  head/x11-wm/obmenu/Makefile
Comment 8 commit-hook freebsd_committer freebsd_triage 2015-07-17 15:54:26 UTC
A commit references this bug:

Author: riggs
Date: Fri Jul 17 15:53:55 UTC 2015
New revision: 392383
URL: https://svnweb.freebsd.org/changeset/ports/392383

Log:
  MFH: r392381

  Fix runtime error:
  hardcoded signal mapping causes openbox to quit while saving

  PR:		201211
  Submitted by:	glolol1@hotmail.com
  Reviewed by:	horia@racoviceanu.com (maintainer)
  Approved by:	ports-secteam (feld) horia@racoviceanu.com (maintainer)

Changes:
_U  branches/2015Q3/
  branches/2015Q3/x11-wm/obmenu/Makefile