On 10/11/2013 3:25 PM, Diego Biurrun wrote: > That mail only refers to "warnings" and "not working". I have yet to see > the actual warnings pasted or get confirmation that the stripping either > works as expected or has no effect on those platforms.
If you looked at the actual code, it was a '-' in front of it, which allows it to not fail. strip will be a no-op when it doesn't recognize -wN because it prints it's default help (like 3 lines). >> strip is not POSIX. strip is *only* binutils. > > http://pubs.opengroup.org/onlinepubs/9699919799/utilities/strip.html This is not actually in POSIX yet. Note "(DEVELOPMENT)". >> The only "portability" check we need is for a sufficiently new >> version. Technically, we don't even need that, since as it is, if -wN >> is not recognized, it is a noop, which is the correct behavior for our >> use case. We are *only* stirpping the spurious labels, and nothing >> else, if -wN exists. > > If indeed nothing happens on these platforms, arguably the correct thing > is to skip stripping entirely. I already asked if we should do this in the other thread I linked. - Derek _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
