Hey Roman,
I just saw this commit:
http://pure-data.svn.sourceforge.net/viewvc/pure-data/trunk/externals/Makefi...
.la files generally should not be installed, that makes sense to me. This seems like a heavy hand to do a global deletion for something like this. The problem would be in the build system that is installing them, and should be fixed there. Then it'll be fixed wherever that library is used, not just in Pd-extended.
.hc
----------------------------------------------------------------------------
Using ReBirth is like trying to play an 808 with a long stick. - David Zicarelli
Hi!
On Thu, Sep 29, 2011 at 11:15:20AM -0400, Hans-Christoph Steiner wrote:
I just saw this commit:
http://pure-data.svn.sourceforge.net/viewvc/pure-data/trunk/externals/Makefi...
.la files generally should not be installed, that makes sense to me.
Actually, in general they should be installed, because libtool uses them when it links against these libraries.
This point is moot if they are only loadable modules, of course.
I just wanted to mention that in general, they should be installed :) Thomas
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 2011-09-29 17:15, Hans-Christoph Steiner wrote:
Hey Roman,
I just saw this commit:
http://pure-data.svn.sourceforge.net/viewvc/pure-data/trunk/externals/Makefi...
.la files generally should not be installed, that makes sense to me. This seems like a heavy hand to do a global deletion for something like this. The problem would be in the build system that is installing them, and should be fixed there. Then it'll be fixed wherever that library is used, not just in Pd-extended.
libtool is convinced that you need .la files. from their perspective it is probably correct.
if you can come up with a proper way to avoid installing .la files from within automake/libtool (without dumping automake/libtool), feel free to suggest fixes. else you will either have to live with those files or to delete them in some other way (e.g. with the one i proposed to roman, and which is currently in externals/Makefile)
probably it is a safer guess to only delete .la files if there is a matching .$(EXTENSION) file and/or a matching .so/.dll file.
nevertheless, i don't see any harm done now.
fgmasdr IOhannes
On Sep 29, 2011, at 11:37 AM, IOhannes m zmoelnig wrote:
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 2011-09-29 17:15, Hans-Christoph Steiner wrote:
Hey Roman,
I just saw this commit:
http://pure-data.svn.sourceforge.net/viewvc/pure-data/trunk/externals/Makefi...
.la files generally should not be installed, that makes sense to me. This seems like a heavy hand to do a global deletion for something like this. The problem would be in the build system that is installing them, and should be fixed there. Then it'll be fixed wherever that library is used, not just in Pd-extended.
libtool is convinced that you need .la files. from their perspective it is probably correct.
if you can come up with a proper way to avoid installing .la files from within automake/libtool (without dumping automake/libtool), feel free to suggest fixes. else you will either have to live with those files or to delete them in some other way (e.g. with the one i proposed to roman, and which is currently in externals/Makefile)
probably it is a safer guess to only delete .la files if there is a matching .$(EXTENSION) file and/or a matching .so/.dll file.
nevertheless, i don't see any harm done now.
What is the problem this is trying to solve? How are the .la files causing problems ? The Pd-extended is crufty enough as it is with lots of kludges like this that I'm mostly responsible for. I've learned thru the error of my ways, and have been working to make it all much cleaner and simpler.
.hc
----------------------------------------------------------------------------
I hate it when they say, "He gave his life for his country." Nobody gives their life for anything. We steal the lives of these kids. - Admiral Gene LeRocque
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 2011-09-29 17:41, Hans-Christoph Steiner wrote:
What is the problem this is trying to solve? How are the .la files causing problems ? The Pd-extended is crufty enough as it is with lots of kludges like this that I'm mostly responsible for. I've learned thru the error of my ways, and have been working to make it all much cleaner and simpler.
roman asked why zexy installs a large number of never-before-seen files, and i told him that they can safely be deleted.
so the "problem" is that they are pointless files, lying around on the filesystem doing no good, and only lead to confusion.
if you prefer to have the files around, feel free to keep them.
mfgasdr IOhannes
PS: btw, debian strongly recommends to remove .la files from packages, e.g. http://wiki.debian.org/ReleaseGoals/LAFileRemoval and a number of lintian warnings.
On Sep 29, 2011, at 11:52 AM, IOhannes m zmoelnig wrote:
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 2011-09-29 17:41, Hans-Christoph Steiner wrote:
What is the problem this is trying to solve? How are the .la files causing problems ? The Pd-extended is crufty enough as it is with lots of kludges like this that I'm mostly responsible for. I've learned thru the error of my ways, and have been working to make it all much cleaner and simpler.
roman asked why zexy installs a large number of never-before-seen files, and i told him that they can safely be deleted.
so the "problem" is that they are pointless files, lying around on the filesystem doing no good, and only lead to confusion.
if you prefer to have the files around, feel free to keep them.
I prefer to not have global kludges in the build system. Yes there are still many, but we should be reducing them not adding them. I don't really care either way about the .la files.
If zexy is causing this problem, then this find/rm thing should be part of the zexy_install target, instead of global.
.hc
----------------------------------------------------------------------------
If you are not part of the solution, you are part of the problem.
On Thu, 2011-09-29 at 12:00 -0400, Hans-Christoph Steiner wrote:
On Sep 29, 2011, at 11:52 AM, IOhannes m zmoelnig wrote:
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 2011-09-29 17:41, Hans-Christoph Steiner wrote:
What is the problem this is trying to solve? How are the .la files causing problems ? The Pd-extended is crufty enough as it is with lots of kludges like this that I'm mostly responsible for. I've learned thru the error of my ways, and have been working to make it all much cleaner and simpler.
roman asked why zexy installs a large number of never-before-seen files, and i told him that they can safely be deleted.
so the "problem" is that they are pointless files, lying around on the filesystem doing no good, and only lead to confusion.
if you prefer to have the files around, feel free to keep them.
I prefer to not have global kludges in the build system. Yes there are still many, but we should be reducing them not adding them. I don't really care either way about the .la files.
If zexy is causing this problem, then this find/rm thing should be part of the zexy_install target, instead of global.
Fair enough. And sorry for adding another global kludge.
IOhannes and me agreed that those files generally are unnecessary. That's why I put the line to the global install target. However, I very much understand your preference not to add more of those global 'solutions'.
If that's ok with you, I'll move it to zexy_install and make it a zexy only solution.
Roman
On Sep 29, 2011, at 3:10 PM, Roman Haefeli wrote:
On Thu, 2011-09-29 at 12:00 -0400, Hans-Christoph Steiner wrote:
On Sep 29, 2011, at 11:52 AM, IOhannes m zmoelnig wrote:
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 2011-09-29 17:41, Hans-Christoph Steiner wrote:
What is the problem this is trying to solve? How are the .la files causing problems ? The Pd-extended is crufty enough as it is with lots of kludges like this that I'm mostly responsible for. I've learned thru the error of my ways, and have been working to make it all much cleaner and simpler.
roman asked why zexy installs a large number of never-before-seen files, and i told him that they can safely be deleted.
so the "problem" is that they are pointless files, lying around on the filesystem doing no good, and only lead to confusion.
if you prefer to have the files around, feel free to keep them.
I prefer to not have global kludges in the build system. Yes there are still many, but we should be reducing them not adding them. I don't really care either way about the .la files.
If zexy is causing this problem, then this find/rm thing should be part of the zexy_install target, instead of global.
Fair enough. And sorry for adding another global kludge.
IOhannes and me agreed that those files generally are unnecessary. That's why I put the line to the global install target. However, I very much understand your preference not to add more of those global 'solutions'.
If that's ok with you, I'll move it to zexy_install and make it a zexy only solution.
Roman
Why not make it part of zexy's build system? If zexy shouldn't install them anyway, that seems like the best place for the fix. Debian's pd-zexy also should not include those files, so it would be fixing that problem also.
.hc
----------------------------------------------------------------------------
kill your television
-
Hans-Christoph Steiner -
IOhannes m zmoelnig -
Roman Haefeli -
Thomas Klausner