Bug 275796 - Register bsdconfig as an application in x11/xdialog
Summary: Register bsdconfig as an application in x11/xdialog
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Larry Rosenman
URL: https://github.com/khorben/freebsd-po...
Keywords:
Depends on:
Blocks:
 
Reported: 2023-12-17 04:22 UTC by Pierre Pronchery
Modified: 2024-01-26 22:57 UTC (History)
2 users (show)

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


Attachments
[PATCH 1/2] x11/xdialog: address warnings from portlint -A (969 bytes, patch)
2023-12-17 04:22 UTC, Pierre Pronchery
no flags Details | Diff
[PATCH 2/2] x11/xdialog: register bsdconfig as an application (10.32 KB, patch)
2023-12-17 04:23 UTC, Pierre Pronchery
no flags Details | Diff
Screenshot from the MATE desktop environment (28.37 KB, image/png)
2023-12-17 04:23 UTC, Pierre Pronchery
no flags Details
Screenshot from the XFCE4 desktop environment (34.52 KB, image/png)
2023-12-17 04:24 UTC, Pierre Pronchery
no flags Details
bsdconfig-48.png (5.09 KB, image/png)
2024-01-23 13:35 UTC, Pierre Pronchery
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Pierre Pronchery 2023-12-17 04:22:20 UTC
Created attachment 247091 [details]
[PATCH 1/2] x11/xdialog: address warnings from portlint -A

Since bsdconfig may be presumed as always available, and uses xdialog
when invoked for use as a graphical interface (ie with -X), this package
may just as well expose bsdconfig to the end user. This also installs a
scaled-down copy of the FreeBSD logo from /boot/images for use as its
icon (48 pixels tall and wide), where the trademark sign was removed
since it is unrecognizable as this scale. Finally, this also documents
the requirements in pkg-message. (Running as root or being a sudoer)
Comment 1 Pierre Pronchery 2023-12-17 04:23:02 UTC
Created attachment 247092 [details]
[PATCH 2/2] x11/xdialog: register bsdconfig as an application
Comment 2 Pierre Pronchery 2023-12-17 04:23:40 UTC
Created attachment 247093 [details]
Screenshot from the MATE desktop environment
Comment 3 Pierre Pronchery 2023-12-17 04:24:00 UTC
Created attachment 247094 [details]
Screenshot from the XFCE4 desktop environment
Comment 4 Pierre Pronchery 2023-12-17 04:25:12 UTC
Since bsdconfig may be presumed as always available, and uses xdialog when invoked for use as a graphical interface (ie with -X), this package may just as well expose bsdconfig to the end user. This also installs a scaled-down copy of the FreeBSD logo from /boot/images for use as its icon (48 pixels tall and wide), where the trademark sign was removed since it is unrecognizable as this scale. Finally, this also documents the requirements in pkg-message. (Running as root or being a sudoer)
Comment 5 Larry Rosenman freebsd_committer freebsd_triage 2023-12-17 19:41:25 UTC
How did this get assigned to me?  
I don't know squat about xdialog.

Re-assigning to ports@
Comment 6 Mark Linimon freebsd_committer freebsd_triage 2023-12-29 04:57:13 UTC
Comment on attachment 247091 [details]
[PATCH 1/2] x11/xdialog: address warnings from portlint -A

^Triage: convert this to text/plain and set the Patch flag so that the
automation can see it.
Comment 7 Larry Rosenman freebsd_committer freebsd_triage 2024-01-21 21:16:10 UTC
@Pierre:
    I tried to apply this, and the png did NOT get created.  Can you fix?
Comment 8 Pierre Pronchery 2024-01-23 13:35:11 UTC
Created attachment 247888 [details]
bsdconfig-48.png
Comment 9 Pierre Pronchery 2024-01-23 13:36:17 UTC
(In reply to Larry Rosenman from comment #7)
Hi Larry,
I guess I simply forgot to add the PNG file to the patch before uploading it here.
I have just added it to this bug report as an attachment.
Comment 10 commit-hook freebsd_committer freebsd_triage 2024-01-26 22:56:48 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=a2e8585ffdf110d5fcc05fa54b7524380a7d9afe

commit a2e8585ffdf110d5fcc05fa54b7524380a7d9afe
Author:     Larry Rosenman <ler@FreeBSD.org>
AuthorDate: 2024-01-26 22:53:58 +0000
Commit:     Larry Rosenman <ler@FreeBSD.org>
CommitDate: 2024-01-26 22:56:22 +0000

    x11/xdialog: Register bsdconfig as an application in x11/xdialog

    Since bsdconfig may be presumed as always available, and uses xdialog
    when invoked for use as a graphical interface (ie with -X), this package
    may just as well expose bsdconfig to the end user. This also installs a
    scaled-down copy of the FreeBSD logo from /boot/images for use as its
    icon (48 pixels tall and wide), where the trademark sign was removed
    since it is unrecognizable as this scale. Finally, this also documents
    the requirements in pkg-message. (Running as root or being a sudoer)

    PR:             275796
    Reported by:    khorben@defora.org

 x11/xdialog/Makefile                      |  13 +++++++++++--
 x11/xdialog/files/bsdconfig-48.png (new)  | Bin 0 -> 5209 bytes
 x11/xdialog/files/bsdconfig.desktop (new) |  12 ++++++++++++
 x11/xdialog/pkg-message (new)             |   5 +++++
 x11/xdialog/pkg-plist                     |   3 +++
 5 files changed, 31 insertions(+), 2 deletions(-)
Comment 11 Larry Rosenman freebsd_committer freebsd_triage 2024-01-26 22:57:33 UTC
Committed, thanks!