Issue 22159 - Check for GNU versions of patch and cp in configure
Summary: Check for GNU versions of patch and cp in configure
Status: CLOSED FIXED
Alias: None
Product: Build Tools
Classification: Code
Component: configure (show other issues)
Version: OOo 1.1
Hardware: All All
: P4 Trivial (vote)
Target Milestone: OOo 2.0
Assignee: rene
QA Contact: issues@tools
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-11-05 19:02 UTC by pavel
Modified: 2006-03-14 21:02 UTC (History)
3 users (show)

See Also:
Issue Type: TASK
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description pavel 2003-11-05 19:02:02 UTC
Right now, we need GNU patch for applying patches and GNU cp for building odk
project.

On some systems (Solaris and FreeBSD) the tools are not compatible (GNU patch
can not apply some patches from the build tree, like the one from boost and
inside odk project, cp -u is used).

Thus we have to check for GNU version of patch and cp in configure. There
already are --with-gnu-patch and --with-gnu-cp.
Comment 1 pavel 2003-11-05 19:02:27 UTC
set target to 2.0.
Comment 2 foskey 2004-05-07 13:31:13 UTC
will patch --version work?

cp --version does not give us GNU but it does give us:

Copyright (C) 2003 Free Software Foundation, Inc.

So we could sense the presence of this.  Just honouring the --version probably
indicates that it is correct.

What are your thoughts.
Comment 3 pavel 2004-05-22 10:00:09 UTC
FreeBSD:

pavel@leda:~> patch --version
Patch version 2.1
pavel@leda:~> gpatch --version
patch 2.5.4
Copyright 1984-1988 Larry Wall
Copyright 1989-1999 Free Software Foundation, Inc.

This program comes with NO WARRANTY, to the extent permitted by law.
You may redistribute copies of this program
under the terms of the GNU General Public License.
For more information about these matters, see the file named COPYING.

written by Larry Wall and Paul Eggert

But gpatch is probably manually installed.
Comment 4 foskey 2004-06-21 02:25:16 UTC
There was a comment to say that under some platforms the option that fails
"standard" cp was removed and the process works fine.  If this is the case then
remove the option and this whole call is redundant.
Comment 5 pavel 2004-09-18 23:06:12 UTC
waratah: should we reassign to rodarvus?
Comment 6 foskey 2004-09-18 23:22:18 UTC
Reassigning.
Comment 7 pavel 2004-11-16 20:18:20 UTC
.
Comment 8 rene 2005-01-13 22:30:47 UTC
Reassign to me
Comment 9 rene 2005-01-13 22:32:51 UTC
committed to cws_src680_configure4
Comment 10 rene 2005-01-14 02:13:43 UTC
set to FIXED
Comment 11 quetschke 2005-01-23 21:01:03 UTC
seen in configure4
Comment 12 rene 2005-03-31 14:54:45 UTC
close