libjava build failure with read-only source tree

Thu Sep 29 09:48:00 GMT 2011

>>> On 29.09.11 at 11:31, Bryce McKinlay <bmckinlay@gmail.com> wrote:
> On Thu, Sep 29, 2011 at 9:18 AM, Jan Beulich <JBeulich@suse.com> 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?

Perhaps to retain the timestamps of the files without becoming too
system specific (i.e. dependent on cp's support of
--preserve=timestamps)?
> In any case, I'd say you can go ahead and check this in under the
> "obvious" rule.

Yeah, probably.
Thanks, Jan