libpng-1.4.0beta4

The semicolon is missing from this line in Makefile.am,
Makefile.in

 echo "#define PNG_NO_MMX_CODE" >>
pngconf.h; \

The consequence is that "configure" fails to
insert the definition when
it is needed, and later on the assembly of pnggccrd.c fails.

Glenn


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web
services, security?
Get stuff done quickly with pre-integrated technology to
make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on
Apache Geronimo
http://sel.as-us.falkag.net/
sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
png-mng-implement mailing list
png-mng-implementlists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/png-m
ng-implement

I received a request to remove the square brackets from
line 33 of configure.ac which currently reads:  

AM_CONFIG_HEADER([config.h])

On ubuntu 5.10, autogen.sh says
configure.ac: 33: required file `./[config.h].in' not found

Are the brackets actually required on any platform?

Glenn



-------------------------------------------------------
Using Tomcat but need to do more? Need to support web
services, security?
Get stuff done quickly with pre-integrated technology to
make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on
Apache Geronimo
http://sel.as-us.falkag.net/
sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
png-mng-implement mailing list
png-mng-implementlists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/png-m
ng-implement

On Fri, May 12, 2006 at 03:09:37PM -0400, Glenn
Randers-Pehrson wrote:
> I received a request to remove the square brackets from
> line 33 of configure.ac which currently reads:  
> 
> AM_CONFIG_HEADER([config.h])
> 
> On ubuntu 5.10, autogen.sh says
> configure.ac: 33: required file `./[config.h].in' not
found
> 
> Are the brackets actually required on any platform?

I think the (now obsoleted)
	AM_CONFIG_HEADER(config.h)
doesn't need brackets, while the currently recommended
	AC_CONFIG_HEADERS([config.h])
does.

Cheers,
 Thomas


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web
services, security?
Get stuff done quickly with pre-integrated technology to
make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on
Apache Geronimo
http://sel.as-us.falkag.net/
sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
png-mng-implement mailing list
png-mng-implementlists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/png-m
ng-implement

At 12:20 PM 5/13/2006 +0200, Thomas Klausner wrote:
>On Fri, May 12, 2006 at 03:09:37PM -0400, Glenn
Randers-Pehrson wrote:
>> I received a request to remove the square brackets
from
>> line 33 of configure.ac which currently reads:  
>> 
>> AM_CONFIG_HEADER([config.h])
>> 
>> On ubuntu 5.10, autogen.sh says
>> configure.ac: 33: required file `./[config.h].in'
not found
>> 
>> Are the brackets actually required on any platform?
>
>I think the (now obsoleted)
>	AM_CONFIG_HEADER(config.h)
>doesn't need brackets, while the currently recommended
>	AC_CONFIG_HEADERS([config.h])
>does.

We had some previous discussion about that:

>Date: Sat, 15 Apr 2006 00:22:00 -0700
>
>From: Ralph Giles
>>I notice you already have the older form
AM_CONFIG_HEADER([config.h])
>>in configure.ac. You can replace that line with the
above, (assuming new 
>>enough automake) or just change the argument.
>
>That's not a good idea.  I did this on the devio
(sourceforge) project and
>in the end had to revert it to AM_CONFIG_HEADER.  This
is compatible with
>old and new automake, whereas the AM_CONFIG_HEADERS
version is not.
>
>John Bowler <jbowleracm.org>
>

Glenn


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web
services, security?
Get stuff done quickly with pre-integrated technology to
make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on
Apache Geronimo
http://sel.as-us.falkag.net/
sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
png-mng-implement mailing list
png-mng-implementlists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/png-m
ng-implement

From: Glenn Randers-Pehrson
>configure.ac: 33: required file `./[config.h].in' not
found
>Are the brackets actually required on any platform?

Out of about 15 uses of that macro that I found only one (in
file-4.13)
uses
the square brackets.  This was by searching an OpenEmbedded
build, so
there
are some fairly fundamental packages there, such as binutils
(2.16) and
gcc 3.4.4

John Bowler <jbowleracm.org>



-------------------------------------------------------
Using Tomcat but need to do more? Need to support web
services, security?
Get stuff done quickly with pre-integrated technology to
make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on
Apache Geronimo
http://sel.as-us.falkag.net/
sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
png-mng-implement mailing list
png-mng-implementlists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/png-m
ng-implement