Compound Literal - xlc and gcc differences can be patched
Since I so miserably misspelled packaging - a new thread specific to the issue at hand.
I found a "workaround". In short, xlc does not accept arrays of nested array - the bottom line (best message) is: 1506-995 (S) An aggregate containing a flexible array member cannot be used as a member of a structure or as an array element.
At the core - all the other messages come because this (I shall call it nesting).
For the test-http-auth.c file I have a patch (attached).
make ran for while with no issue, then it started running into several files with this same style of specification - doveadm-dict.c being the first one. I ran "make -i" and I notice several files with this - for xlc - "unaccepted" syntax.
Please look at the patch - what it basically is - and let me know whether you would consider accepting patches in this form (e.g., verify gcc (I assume) will accept it as well). If your answer is yes, I shall proceed with the additional files (learn how your MACROS work!) and send them to you. Again, initially I would just send one, e.g., doveadm-dict.c - to be sure your regular compiler also builds this alternate specification.
Thank you for your consideration!
Michael
On 11.10.2016 23:25, Michael Felt wrote:
Since I so miserably misspelled packaging - a new thread specific to the issue at hand.
I found a "workaround". In short, xlc does not accept arrays of nested array - the bottom line (best message) is: 1506-995 (S) An aggregate containing a flexible array member cannot be used as a member of a structure or as an array element.
At the core - all the other messages come because this (I shall call it nesting).
For the test-http-auth.c file I have a patch (attached).
make ran for while with no issue, then it started running into several files with this same style of specification - doveadm-dict.c being the first one. I ran "make -i" and I notice several files with this - for xlc - "unaccepted" syntax.
Please look at the patch - what it basically is - and let me know whether you would consider accepting patches in this form (e.g., verify gcc (I assume) will accept it as well). If your answer is yes, I shall proceed with the additional files (learn how your MACROS work!) and send them to you. Again, initially I would just send one, e.g., doveadm-dict.c - to be sure your regular compiler also builds this alternate specification.
Thank you for your consideration!
Michael
Hi!
Please make your patch, if possible, via https://github.com/dovecot/core as pull request.
Aki Tuomi Dovecot oy
On 12/10/2016 07:51, Aki Tuomi wrote:
Please make your patch, if possible, viahttps://github.com/dovecot/core as pull request.
I am not really git "schooled", but I shall look into that.
Many Thanks for being open to a "non-gcc" compiler!
Michael
Hi,
As preparations for a port to AIX - making sure I can get it to build on linux with gcc. While it is fairly easy to add the requirements (on GNU Linux) having pandoc as a configure "hardstop", even with --without-docs or --with-docs=no is a bit worrisome as I am fearful that I amy not be able to get pandoc ported as well.
Hence a request that a missing pandoc become a warnig, and just not done when pandoc is not available, or at least accept one of --without-docs and --with-docs=no (or should I be using --with-pandata=no? )
Michael
On November 19, 2016 at 4:01 PM Michael Felt michael@felt.demon.nl wrote:
Hi,
As preparations for a port to AIX - making sure I can get it to build on linux with gcc. While it is fairly easy to add the requirements (on GNU Linux) having pandoc as a configure "hardstop", even with --without-docs or --with-docs=no is a bit worrisome as I am fearful that I amy not be able to get pandoc ported as well.
Hence a request that a missing pandoc become a warnig, and just not done when pandoc is not available, or at least accept one of --without-docs and --with-docs=no (or should I be using --with-pandata=no? )
Michael
At the moment, pandoc is only *required* if you are building from git. And it's not even required then, just do env PANDOC=true ./configure ...
Aki
On 19/11/2016 15:09, Aki Tuomi wrote:
Michael At the moment, pandoc is only*required* if you are building from git. And it's not even required then, just do env PANDOC=true ./configure ...
Aki
Thanks! guess I should have read ./configure --help more closely.
So, is there something else I have forgotten - in order to get a good summary of the results of 'make check'?
Michael
participants (2)
-
Aki Tuomi
-
Michael Felt