Hi Chase,

On 6/6/26 22:38, Chase via cdesktopenv-devel wrote:
Hello all,
I've been working on this patch for some time. It seems like there are a few high churn commits coming soon, so I want to get this into the code before I need to rewrite a bunch of things. This reintroduces initial support for AIX 7.2 and above, as well as ensuring 64 bit support for them. Support for building documentation is not yet finished, so configure should be run with --disable-docs. I also could not test dtlogin on my setup due to needing X11 forwarding + Xephyr for reasons I won't go into, which required display :1 and dtlogin seems to be hardcoded for :0.


I try to address merge requests on a first-come first-served basis, so I have already merged some things today, like Chinese locale support, among others.

There is also a ticket (https://sourceforge.net/p/cdesktopenv/tickets/200/) that allows the install locations (/usr/dt, /var/dt, etc) to be overridden properly, probably doing some of the things you mention have in your patch.

I want the patches in ticket #200 merged first, once there has been some time for me (and others hopefully) to try them out.

As for your patch... I really appreciate the effort it takes to add support for a new (or old, in this case) platform.  Having done it, I know it is not a trivial task.

However, this patch is simply unreviewable/unmergable...

It's nearly 30K lines long and touches just about everything in some fashion, and tries to do way to much for a single patch or patch-set.  There is just no way I can apply this and just "hope for the best" :)

This needs to be sub-divided into separate tasks with patches or patch-sets that accomplish that specific task and build on previous merged patches (ie: current master).

For example, getting rid of tradcpp, though I still don't know how you expect to handle conditionals with sed - but I digress. That is a task you could do - get that merged first, then move on to the next task, etc.

That way you don't try to do it all at once, and your patches are confined to specific tasks and are small enough to be reviewed by mere humans with limited time. :)

Also, there is no reason you cannot use gmake on AIX - the BSDs do this already and it works fine.

I do not think changes there are a "requirement" for AIX support, right? Then don't do it.

Or if you must - make that a task to get reviewed/merged as a standalone (!) patch into a current version of master.  But be prepared if it does not pass muster.

I hope you see what points I am getting at here...

-jon




Here is my workflow for building (this assumes your AIX machine is already up and running properly with internet connection and DNF set up):

  * Install roughly these packages: gcc gcc-c++ gmake (if you want
    docs) autoconf automake libtool git \

            pkg-config flex pcre2-devel libXcursor-devel libXrender-devel libXft-devel   \
            fontconfig-devel harfbuzz-devel libpng-devel libjpeg-devel  \
            check-devel aix-x11-pc gzip libXau-devel libXdmcp-devel

  * Build from source the following: liblmdb, onsgmls (the openjade
    repo on sourceforge will not build, I used this one
    <https://github.com/deepin-community/opensp> instead), and motif
    2.5.2 <https://github.com/thentenaar/motif>.

  * export OBJECT_MODE=64

  * ./autogen.sh

  * NM="nm -X64 -B" AR="ar -X64" ./configure --disable-xinerama
    --disable-docs --build=powerpc-ibm-aix7.2.0.0
    --with-tcl=/opt/freeware/lib64 M4="/opt/freeware/bin/m4"
    LDFLAGS="-lpthread" CFLAGS="-maix64"

  * make
  * make install


And that should do it. This patch contains a lot of things:

  * All gnuisms in the non-doc Makefile.am's have been removed to
    enable non gmake makes to build. This involved my previous effort
    to move many scripts built with GENCPP to instead use the
    autotools to do sed replacements.

  * Xinerama can now be toggled with --disable-xinerama, as the base
    X11 server on AIX does not support Xinerama.
  * A large amount of hardcoded /usr/dt, /var/dt and /etc/dt entries
    have been replaced with their proper variables in scripts.

  * Lots of needless old AIX cruft pruned.


Let me know if there are any issues with the patch on platforms other than AIX 7.2, as I only tested it to make sure it ran there, and I am having trouble getting shared folders to work in UTM to test elsewhere. I have attached a screenshot of it successfully running. Once this is working fully, I would recommend any that if there are any enterprises reading this that still requiring CDE on AIX, to switch to this one, as the one currently packaged by IBM on AIX is CDE 1.0.

Thank you for your time,
-Chase


_______________________________________________
cdesktopenv-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel

--
Jon Trulson

  "The less you know, the more you believe."
                           -- Bono
_______________________________________________
cdesktopenv-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel

Reply via email to