Closed
Bug 330142
Opened 19 years ago
Closed 19 years ago
Building of minimo by Mozilla_Build_FAQ failed
Categories
(Minimo Graveyard :: Build Config, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: romaxa, Assigned: dougt)
References
Details
Attachments
(1 file)
297 bytes,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.12) Gecko/20050922 Firefox/1.0.7 (Debian package 1.0.7-1)
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.12) Gecko/20050922 Firefox/1.0.7 (Debian package 1.0.7-1)
Building Minimo browser, using simple instuctions from
http://developer.mozilla.org/en/docs/Mozilla_Build_FAQ
failed.
Reproducible: Always
Steps to Reproduce:
1. cvs -z3 -d :pserver:anonymous@cvs-mirror.mozilla.org:/cvsroot co mozilla/client.mk
2. cp mozilla/minimo/config/mozconfig/linux_x86 mozilla/mozconfig
3. make -f client.mk checkout
autoconf && ./configure && make
Actual Results:
Building failed with message:
make[5]: Leaving directory `/home/work/mozilla_new/mozilla/intl/uconv/native'
make[5]: Entering directory `/home/work/mozilla_new/mozilla/intl/uconv/src'
make[5]: *** No rule to make target `../ucvnative/libucnative_s.a', needed by `libuconv.a'. Stop.
Expected Results:
Building finished
I have found 2 solution for this problem:
1)
--- mozilla/configure.in.orig 2006-03-11 11:59:14.000000000 +0200
+++ mozilla/configure.in 2006-03-11 11:57:40.000000000 +0200
@@ -4405,7 +4405,7 @@
MOZ_SVG=
MOZ_UNIVERSALCHARDET=
MOZ_UPDATER=
- MOZ_USE_NATIVE_UCONV=1
+ MOZ_USE_NATIVE_UCONV=
MOZ_V1_STRING_ABI=
MOZ_VIEW_SOURCE=
MOZ_XPFE_COMPONENTS=
2)
1. echo "ac_add_options --disable-native-uconv" >> mozilla/minimo/config/mozconfig/linux_x86
2. cp mozilla/minimo/config/mozconfig/linux_x86 mozilla/mozconfig
Assignee | ||
Comment 1•19 years ago
|
||
this is a bandaide. on linux we really should use native uconv.
Can you see why the libary names are different?
Status: UNCONFIRMED → NEW
Ever confirmed: true
Reporter | ||
Comment 2•19 years ago
|
||
BUGFIX:
Has been fixed wrong directory in Makefile.
Reporter | ||
Comment 3•19 years ago
|
||
(In reply to comment #2)
> Created an attachment (id=215002) [edit]
> BUGFIX
>
> BUGFIX:
>
> Has been fixed wrong directory in Makefile.
>
it is the third, and I think correct solution ;)
Comment 4•19 years ago
|
||
Comment on attachment 215002 [details] [diff] [review]
BUGFIX:
wrong reference at old directory. intl/ucinv/ucnative
It would help Doug and I if you marked the review?dougt@meer.net or review?benjamin@smedbergs.us flag on these patches (it would also be good to name the attachment something more descriptive than BUGFIX so that we know what it is when we're looking at our review queue).
Attachment #215002 -
Flags: review+
Updated•19 years ago
|
Whiteboard: [checkin needed]
Reporter | ||
Comment 5•19 years ago
|
||
Comment on attachment 215002 [details] [diff] [review]
BUGFIX:
wrong reference at old directory. intl/ucinv/ucnative
intl/uconv directory don't contain "ucnative" folder anymore (instead it - "native"), but Makefile.in in has reference on old directory name
Attachment #215002 -
Attachment description: BUGFIX → BUGFIX:
Reporter | ||
Updated•19 years ago
|
Attachment #215002 -
Attachment description: BUGFIX: → BUGFIX:
wrong reference at old directory. intl/ucinv/ucnative
Comment 6•19 years ago
|
||
was this patch check in Doug or Benjamin ?
Romaxa, shouldn't it be marked as duplicated of 213325 or vice-versa ?
Comment 7•19 years ago
|
||
(In reply to comment #6)
> was this patch check in Doug or Benjamin ?
>
> Romaxa, shouldn't it be marked as duplicated of 213325 or vice-versa ?
>
ops, not 213325 but 328736 ! Sorry
Assignee | ||
Comment 8•19 years ago
|
||
Checking in Makefile.in;
/cvsroot/mozilla/intl/uconv/src/Makefile.in,v <-- Makefile.in
new revision: 1.86; previous revision: 1.85
done
fixed on trunk. the branch does NOT have this problem.
marking fixed.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 9•19 years ago
|
||
*** Bug 328736 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•