Skip to content

motif: New port 2.5.2#32283

Open
thentenaar wants to merge 2 commits intomacports:masterfrom
thentenaar:motif
Open

motif: New port 2.5.2#32283
thentenaar wants to merge 2 commits intomacports:masterfrom
thentenaar:motif

Conversation

@thentenaar
Copy link
Copy Markdown

Description

As the original ICS upstream is long dead, I have taken up the mantle of maintaining and improving Motif. The name change is for consistency and clarity (even ICS had dropped the 'open' prefix some years back). I'll volunteer to maintain the port for now, as the original is unmaintained, unless someone else really wants to do so.

Type(s)

New port (deprecating an old port).

  • bugfix
  • enhancement
  • security fix
Tested on

macOS 11.7.11 20G1443 x86_64
Command Line Tools 13.2.0.0.1.1638488800

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL in commit message?
  • checked your Portfile with port lint?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?
  • checked that the Portfile's most important variants haven't been broken?

@thentenaar thentenaar changed the title x11/motif: New port x11/motif: New port 2.5.2 Apr 21, 2026
@thentenaar thentenaar changed the title x11/motif: New port 2.5.2 motif: New port 2.5.2 Apr 21, 2026
Comment thread x11/motif/Portfile Outdated
Copy link
Copy Markdown
Contributor

@reneeotten reneeotten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@thentenaar in addition to some inline review comments, you will need to make sure that dependents of the original openmotif port are transitioned to the new one.

❯ port echo depends:'openmotif($|\s)'
aircraft_oap
alliance
arb
ddd
geomview
glw
grace
molmol
ncplot
ncpp
nedit
opendx
SoXt
xastir
xephem
xmfract
xmgr

Comment thread x11/motif/Portfile
github.tarball_from archive
revision 0

name motif
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is set automatically from the github.setup line - please remove

Comment thread x11/motif/Portfile
Group, and other vendors, it lives on as an \
independent project.

homepage https://github.com/thentenaar/motif
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is set automatically from the github.setup line - please remove

Comment thread x11/motif/Portfile
name motif
categories x11
license LGPL-2.1
maintainers @thentenaar
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ideally an (obfuscated) e-mail address is added here as well, and if there is no objection for others to make minor changes to the Portfile please add openmaintainer as well

Comment thread x11/openmotif/Portfile
@@ -1,6 +1,8 @@
# -*- coding: utf-8; mode: tcl; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- vim:fenc=utf-8:ft=tcl:et:sw=4:ts=4:sts=4
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

more clean-up of the Portfile is needed when using the obsolete PortGroup - please see the relevant section in the guide.

Comment thread x11/motif/Portfile
Comment on lines +56 to +59
post-destroot {
set docdir ${prefix}/share/doc/${name}
xinstall -d ${destroot}${docdir}
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this doesn't seem to do anything except for creating an empty directory... or?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants