Re: libjava build failure with read-only source tree

On 09/29/2011 10:31 AM, Bryce McKinlay wrote:
> On Thu, Sep 29, 2011 at 9:18 AM, Jan Beulich <[email protected]> wrote:
> 
>> --- a/libjava/classpath/tools/Makefile.am
>> +++ b/libjava/classpath/tools/Makefile.am
>> @@ -371,6 +371,7 @@ endif
>>  ## BEGIN GCJ LOCAL
>>        cp -pR $(srcdir)/asm .
>>        cp -pR $(srcdir)/classes .
>> +       find asm classes -type d -exec chmod u+w \{\} \;
>>        if [ -d asm/.svn ]; then \
>>          find asm classes -depth -type d -name .svn -exec rm -rf \{\} \;; \
>>        fi
> 
> This patch looks fine to me, though I wonder if there is any good
> reason why "cp -p" was used in the first place?
> 
> In any case, I'd say you can go ahead and check this in under the
> "obvious" rule.

Maybe, but I'm not sure why the files are being copied in the source
tree anyway.  Unless gcj-maintainer-mode is enabled, it shouldn't
be necessary.

Andrew.