Issue 32398 - Repeated build in libxml2 fails
Summary: Repeated build in libxml2 fails
Status: CLOSED FIXED
Alias: None
Product: Build Tools
Classification: Code
Component: code (show other issues)
Version: 680m49
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 2.0
Assignee: pavel
QA Contact: issues@tools
URL:
Keywords:
: 34419 (view as issue list)
Depends on:
Blocks:
 
Reported: 2004-08-01 14:55 UTC by pavel
Modified: 2004-11-29 10:45 UTC (History)
3 users (show)

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


Attachments
Remove config_h from patch file and ADDITIONAL_FILES (8.29 KB, patch)
2004-08-23 09:21 UTC, chris
no flags Details | Diff
add back the needed config files, someone too heavy handed. (497 bytes, patch)
2004-09-20 22:05 UTC, foskey
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this issue.
Description pavel 2004-08-01 14:55:02 UTC
Hi,

repeated build in libxml2 fails:

oo@oo:~/BuildDir/ooo_SRC680_m49_src/libxml2> build
build -- version: 1.113

/home/oo/BuildDir/ooo_SRC680_m49_src/libxml2
-------------
cd ./unxlngi4.pro/misc/build && cat ../../../libxml2-2.5.11.patch | patch -b -p2
&& touch so_patched_so_libxml2
patching file libxml2-2.5.11/config.h
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file libxml2-2.5.11/config.h.rej
patching file libxml2-2.5.11/makefile.mk
Reversed (or previously applied) patch detected!  Assume -R? [n]
Comment 1 pavel 2004-08-01 14:55:24 UTC
set target
Comment 2 chris 2004-08-23 09:21:43 UTC
Created attachment 17290 [details]
Remove config_h from patch file and ADDITIONAL_FILES
Comment 3 chris 2004-08-23 09:30:19 UTC
So, the problem lies with config.h being declared in ADDITIONAL_FILES:

Files in ADDITIONAL_FILES are touched into existence before applying the patch.
 After the files are patched, ADD_FILES_FLAG_FILE (so_addfiles_so_libxml2 in
this case) is touched.  To function in the right order, ADD_FILES_FLAG_FILE must
depend on each file in ADDITIONAL_FILES.

But config.h is a file that is generated by configure.  There is no point in
touching/patching it before configure is run, since configure overwrites the
file.  After configure is run, config.h will be newer than
so_addfiles_so_libxml2, which triggers dmake to try to reapply the patches,
which fails.

haggai->lo:  Please can you explain why you added config.h back in makefile.mk
rev 1.4? The commit message does not explain why or reference an issue.

Patch is attached that removes config.h from additional files again and fixes
the issue (tested on my Linux system).
Comment 4 pavel 2004-08-28 10:51:41 UTC
This patch also applies to SRC680_m52 and works...
Comment 5 pavel 2004-09-09 11:50:45 UTC
I'll take over this issue so it is fixed sooner.
Comment 6 pavel 2004-09-09 12:08:22 UTC
Fixed in pj02.
Comment 7 hjs 2004-09-13 16:56:31 UTC
applying this patch doesn't seem to change the resulting "config.h" :-)
Comment 8 maho.nakata 2004-09-14 09:12:40 UTC
hjs:
can be different when OS is changed.
Comment 9 foskey 2004-09-20 21:58:30 UTC
Build on HEAD fails:

make writeable...
cd ./unxlngi6.pro/misc/build && cat ../../../libxml2-2.5.11.patch | patch -b -p2
&& touch so_patched_so_libxml2
can't find file to patch at input line 3
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|*** misc/libxml2-2.5.11/makefile.mk	Tue May 11 18:05:00 2004
|--- misc/build/libxml2-2.5.11/makefile.mk	Tue May 11 18:02:59 2004
--------------------------
File to patch: 
Skip this patch? [y] 
Skipping patch.
1 out of 1 hunk ignored
can't find file to patch at input line 141
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|*** misc/libxml2-2.5.11/xml2.map	Tue May 11 18:05:01 2004
Comment 10 foskey 2004-09-20 22:05:08 UTC
Created attachment 17817 [details]
add back the needed config files, someone too heavy handed.
Comment 11 pavel 2004-09-21 06:45:14 UTC
as said above, this is fixed in pj02. It is not integrated yet.
Comment 12 pavel 2004-09-21 07:10:24 UTC
*** Issue 34419 has been marked as a duplicate of this issue. ***
Comment 13 pavel 2004-11-29 10:45:18 UTC
verified, closing.