I like what I see ;-)
I haven't looked at the code, but one warning: it's important to not
assume that blah/../foo/ is equivalent to foo/ - blah might be a
symlink.
*Justification (trimpath)*: trimpath can be used to shorten the
input strings to compilers/linkers, as well as improve
Pretty weak. If a few more include paths were added to the project it would
still break, regardless of your patch.
When you have thousands of object files, shortening the name of each
by several bytes when invoking ar can make room for really quite a lot
more files before you run up against
On Tue, 31 May 2011 20:57:10 +0900, Edward Welbourne e...@opera.com
wrote:
I haven't looked at the code, but one warning: it's important to not
assume that blah/../foo/ is equivalent to foo/ - blah might be a
symlink.
While this is true, I am not sure one which side this should be solved.
On Tue, May 31, 2011 at 8:06 AM, Edward Welbourne e...@opera.com wrote:
Never understimate the value of a modest shortening of file names -
when our ar command-line got to about 230 kibibytes, we had to re-work
the way we invoked ar !
My vote would be favorable as well. There are a number of
Edward Welbourne wrote:
Pretty weak. If a few more include paths were added to the project it would
still break, regardless of your patch.
When you have thousands of object files, shortening the name of each
by several bytes when invoking ar can make room for really quite a lot
more files
