Re: [cdesktopenv-devel] [PATCH] [autotools-conversion]

2019-10-23 Thread Jon Trulson
On 10/23/19 4:18 PM, Chase wrote: > Most of it should work in theory™, but the most testing I did was to make > sure that we could get past the configure.ac file, but I made some changes > afterwards. I know what shouldn't work for sure is appbuilder, dthelp and > dtinfo, they use some imake mag

Re: [cdesktopenv-devel] [PATCH] [autotools-conversion]

2019-10-23 Thread Chase via cdesktopenv-devel
Most of it should work in theory™, but the most testing I did was to make sure that we could get past the configure.ac file, but I made some changes afterwards. I know what shouldn't work for sure is appbuilder, dthelp and dtinfo, they use some imake magic that is pretty complex, makedepend also

Re: [cdesktopenv-devel] [PATCH] [autotools-conversion]

2019-10-23 Thread Jon Trulson
On 10/23/19 3:58 PM, Chase wrote: > I mimicked imake in this case, and put an include to a file that uses > variables for a cpp command. There might be a better way to do this (this > might not even work at all), but I was hoping to stick to what works for now > and autotoolize the code as we go

Re: [cdesktopenv-devel] [PATCH] [autotools-conversion]

2019-10-23 Thread Chase via cdesktopenv-devel
I mimicked imake in this case, and put an include to a file that uses variables for a cpp command. There might be a better way to do this (this might not even work at all), but I was hoping to stick to what works for now and autotoolize the code as we go along, the configure.ac also mimicks imak

Re: [cdesktopenv-devel] [PATCH] [autotools-conversion]

2019-10-23 Thread Jon Trulson
On 10/23/19 2:56 PM, Chase via cdesktopenv-devel wrote: > I tried to make these patches as distinct as possible, this constitutes > an estimated 80% of the code covered Nice! I can see that took a long time. As discussed, I have applied these patches to the new autotools-conversion branch. Nice