On Thu, Apr 17, 2008 at 11:27 AM, Theppitak Karoonboonyanan
<[EMAIL PROTECTED]> wrote:
> On Thu, Apr 17, 2008 at 10:34 AM, Paul Wise <[EMAIL PROTECTED]> wrote:
>  >  Looks like this in the upstream makefile:
>  >
>  >  all allbin alldoc tags clean:
>  >         @for I in ${subdirs}; do (cd $$I; ${MAKE} $@ || exit 1); done
>  >
>  >  Should be replaced with this:
>  >
>  >  all allbin alldoc tags clean:
>  >         @for I in ${subdirs}; do (cd $$I; ${MAKE} $@) || exit 1; done
>  >
>  >  Same for the other targets in the Makefile.in that have a similar 
> construct.
>
>  Indeed. Thanks for pointing out.

Done in upstream CVS.

>  For the Xt dependency, it was missed during libSM and libICE removal.
>  Indeed, xt needs to be checked in the configure script, in addition to x11.
>  And hopefully, -Wl,--as-needed can prevent libSM and libICE from being
>  pulled in along with libxt again.

Dependency on X11/Intrinsic.h has been eliminated instead
in upstream CVS.

In fact, it's doubtable that the build only required the
header file, but not the library linkage. That means no
symbols were called, only data types or macros were used.
And fortunately, all can be expressed in terms of pure Xlib.

So, libxt-dev build-dep should not be required in next
upstream version.

Regards,
-- 
Theppitak Karoonboonyanan
http://linux.thai.net/~thep/


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to