From: Alan W. I. <ir...@be...> - 2003-09-03 21:01:27
|
This came up in private discussions with Rafael, and we thought we should get the list involved in the discussion. Alan: > "NOBRAINDEAD appears in a number of places in the source tree, but it is essentially totally unsupported in configure.ac and sysloc.in." Rafael: > "The Right Way (TM) of doing this is by adding an AC_DEFINE(NOBRAINDEAD,...) to configure.ac and having its value being controlled by a configure option like --enable-nobraindead. The current situation is not satisfactory, since plConfig.h.in (which is CVS controlled) must be edited to change this #define. We could issue a *_big_* warning if the user specify that switch." My first tendency is to say just leave it in its current (unsupported-by-autotools) state since it is such an exotic option that is rarely used (never by me, for example). However, I agree with Rafael's implicit point, that having half-supported options around like this can be nothing but a pain for future maintenance/understanding of what is going on with NOBRAINDEAD. I am willing to make the changes (essentially following what is done for the with_freetype logic) if the rest of the list feel this is worthwhile. Or would it be better to remove NOBRAINDEAD completely from the source tree. Has anybody actually used it in the last 5 years? Alan |
From: <jc...@fe...> - 2003-09-11 23:04:01
|
On Wednesday 03 September 2003 22:00, Alan W. Irwin wrote: | This came up in private discussions with Rafael, and we thought we should | get the list involved in the discussion. | | Alan: | > "NOBRAINDEAD appears in a number of places in the source tree, but it is | | essentially totally unsupported in configure.ac and sysloc.in." | | Rafael: | > "The Right Way (TM) of doing this is by adding an | | AC_DEFINE(NOBRAINDEAD,...) to configure.ac and having its value being | controlled by a configure option like --enable-nobraindead. The current | situation is not satisfactory, since plConfig.h.in (which is CVS | controlled) must be edited to change this #define. We could issue a | *_big_* warning if the user specify that switch." | | My first tendency is to say just leave it in its current | (unsupported-by-autotools) state since it is such an exotic option that is | rarely used (never by me, for example). However, I agree with Rafael's | implicit point, that having half-supported options around like this can be | nothing but a pain for future maintenance/understanding of what is going | on with NOBRAINDEAD. | | I am willing to make the changes (essentially following what is done for | the with_freetype logic) if the rest of the list feel this is worthwhile. | Or would it be better to remove NOBRAINDEAD completely from the source | tree. Has anybody actually used it in the last 5 years? Hi, I think this issue was already addressed some time ago. I think to remember that Maurice said that BRAINDEAD was needed by some fortran compilers that some colleagues of him was using. Joao | | Alan | | | ------------------------------------------------------- | This sf.net email is sponsored by:ThinkGeek | Welcome to geek heaven. | http://thinkgeek.com/sf | _______________________________________________ | Plplot-devel mailing list | Plp...@li... | https://lists.sourceforge.net/lists/listinfo/plplot-devel |
From: Alan W. I. <ir...@be...> - 2003-09-13 03:42:07
|
On 2003-09-12 00:05+0100 Jo=E3o Cardoso wrote: > On Wednesday 03 September 2003 22:00, Alan W. Irwin wrote: > | This came up in private discussions with Rafael, and we thought we sho= uld > | get the list involved in the discussion. > | > | Alan: > | > "NOBRAINDEAD appears in a number of places in the source tree, but i= t is > | > | essentially totally unsupported in configure.ac and sysloc.in." > | > | Rafael: > | > "The Right Way (TM) of doing this is by adding an > | > | AC_DEFINE(NOBRAINDEAD,...) to configure.ac and having its value being > | controlled by a configure option like --enable-nobraindead. The curre= nt > | situation is not satisfactory, since plConfig.h.in (which is CVS > | controlled) must be edited to change this #define. We could issue a > | *_big_* warning if the user specify that switch." > | > | My first tendency is to say just leave it in its current > | (unsupported-by-autotools) state since it is such an exotic option tha= t is > | rarely used (never by me, for example). However, I agree with Rafael'= s > | implicit point, that having half-supported options around like this ca= n be > | nothing but a pain for future maintenance/understanding of what is goi= ng > | on with NOBRAINDEAD. > | > | I am willing to make the changes (essentially following what is done f= or > | the with_freetype logic) if the rest of the list feel this is worthwhil= e. > | Or would it be better to remove NOBRAINDEAD completely from the source > | tree. Has anybody actually used it in the last 5 years? > > Hi, > > I think this issue was already addressed some time ago. I think to rememb= er > that Maurice said that BRAINDEAD was needed by some fortran compilers tha= t > some colleagues of him was using. Maurice, what do you think the best option is here? I am still willing to take the 15 minutes to put NOBRAINDEAD under autotools control, but all I could test is whether plConfig.h got changed appropriately by a configure option set up properly by autotools. However, you would probably have to test the actual result of using the option. I have no idea what to expect if a build was actually attempted with NOBRAINDEAD defined or whether it is completely supported any more in the source code. Alan __________________________ Alan W. Irwin email: ir...@be... phone: 250-727-2902 Astronomical research affiliation with Department of Physics and Astronomy, University of Victoria (astrowww.phys.uvic.ca). Programming affiliations with the PLplot scientific plotting software package (plplot.org), the Yorick front-end to PLplot (yplot.sf.net), the Loads of Linux Links project (loll.sf.net), and the Linux Brochure Project (lbproject.sf.net). __________________________ Linux-powered Science __________________________ |
From: Maurice L. <mj...@ga...> - 2003-09-20 20:02:03
|
Alan W. Irwin writes: > Maurice, what do you think the best option is here? I am still willing to > take the 15 minutes to put NOBRAINDEAD under autotools control, but all I > could test is whether plConfig.h got changed appropriately by a configure > option set up properly by autotools. However, you would probably have to > test the actual result of using the option. I have no idea what to expect > if a build was actually attempted with NOBRAINDEAD defined or whether it > is completely supported any more in the source code. In the beginning, there was no BRAINDEAD, and no NOBRAINDEAD. Reasonable systems took fortran external names, lowercased them, and appended an underscore. Then there was AIX, and others even stranger, which forced me into the shenanighens you see with the stuff in bindings/f77/plstubs.h, plus [NO]BRAINDEAD in plplot.h. Frankly, at this point I am resigned to the situation that the C-API BRAINDEAD redefinitions are here for good. So I don't think there's any need to support NOBRAINDEAD any longer. -- Maurice LeBrun Lightspeed Semiconductor Corp |
From: Alan W. I. <ir...@be...> - 2003-09-20 21:42:11
|
On 2003-09-20 15:00-0500 Maurice LeBrun wrote: > Alan W. Irwin writes: > > Maurice, what do you think the best option is here? I am still willing to > > take the 15 minutes to put NOBRAINDEAD under autotools control, but all I > > could test is whether plConfig.h got changed appropriately by a configure > > option set up properly by autotools. However, you would probably have to > > test the actual result of using the option. I have no idea what to expect > > if a build was actually attempted with NOBRAINDEAD defined or whether it > > is completely supported any more in the source code. > > In the beginning, there was no BRAINDEAD, and no NOBRAINDEAD. Reasonable > systems took fortran external names, lowercased them, and appended an > underscore. > > Then there was AIX, and others even stranger, which forced me into the > shenanighens you see with the stuff in bindings/f77/plstubs.h, plus > [NO]BRAINDEAD in plplot.h. > > Frankly, at this point I am resigned to the situation that the C-API BRAINDEAD > redefinitions are here for good. So I don't think there's any need to support > NOBRAINDEAD any longer. OK. It is gone. A test build seemed to go okay so I committed the corresponding changes (plus a change in one relevant paragraph in the documentation.) Note, I have left sys/win-tk (only one reference to NOBRAINDEAD) and sys/win32/msdev (many references) alone. AFAIK, my changes don't affect those trees, but Vince and Olof may want to clean out NOBRAINDEAD from those trees as well. Alan __________________________ Alan W. Irwin email: ir...@be... phone: 250-727-2902 Astronomical research affiliation with Department of Physics and Astronomy, University of Victoria (astrowww.phys.uvic.ca). Programming affiliations with the PLplot scientific plotting software package (plplot.org), the Yorick front-end to PLplot (yplot.sf.net), the Loads of Linux Links project (loll.sf.net), and the Linux Brochure Project (lbproject.sf.net). __________________________ Linux-powered Science __________________________ |