Closed
Bug 292804
Opened 20 years ago
Closed 20 years ago
Unable to build reporter
Categories
(Other Applications Graveyard :: Reporter, defect)
Other Applications Graveyard
Reporter
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: raccettura, Assigned: jay)
References
Details
Attachments
(1 file, 1 obsolete file)
3.91 KB,
patch
|
chase
:
review+
bryner
:
superreview+
asa
:
approval-aviary1.1a1+
asa
:
approval1.8b2+
|
Details | Diff | Splinter Review |
copy
"/cygdrive/c/builds/tinderbox/Fx-Trunk/WINNT_5.0_Depend/mozilla/xpinstall/packager/common/share.t"
reporter.template
cat: reporter.jst: No such file or directory
Making reporter.xpi...
adding: install.js (stored 0%)
This doesn't look right.
Assignee | ||
Comment 1•20 years ago
|
||
bsmedberg: Do you think any of the l10n changes you made in bug 285653 could
have broken something? after my first checking
(https://bugzilla.mozilla.org/attachment.cgi?id=177084) and before i "disabled"
reporter (https://bugzilla.mozilla.org/attachment.cgi?id=179810), things were
ok. it looks like a lot of the reporter directory structure changed for the
localization, could the be the cause? We need to get this figured out asap if
reporter is going to make it into 1.1a.
Comment 2•20 years ago
|
||
Jay, it doesn't look like you are copying the reporter.jst file at
http://lxr.mozilla.org/mozilla/source/browser/installer/windows/Makefile.in#58
In addition, you should remove the locale registration at
http://lxr.mozilla.org/mozilla/source/browser/installer/windows/reporter.jst#13
Updated•20 years ago
|
Flags: blocking1.8b2+
Flags: blocking-aviary1.1+
Assignee | ||
Comment 3•20 years ago
|
||
These are the changes Benjamin suggested, but I am unable to complete a build.
Can someone else try this patch and see if they have any luck? I'll be out of
town 4/5 - 4/8, so if anyone is able to fix this, pleases get reviews and check
it in. Thanks.
Reporter | ||
Comment 4•20 years ago
|
||
bsmedberg: ca you try this patch and see if this fixes us? We really need to
get this going quickly at this point, so any help you can provide is _greatly_
appreciated as always.
--> jay as he's doing the lifting in this bug.
Assignee: robert → jay
Assignee | ||
Comment 5•20 years ago
|
||
I finally got my machine setup for builds and was able to compile a fresh pulled
tree...unfortunately, I'll be out of town until Sunday afternoon. I'll try
making my changes again and trying out a build then...but if someone can test it
out before, that will be great. Thanks.
Comment 6•20 years ago
|
||
I am in the middle of l10n and software update work; I would prefer if somebody
else could fix whatever's wrong here.
Updated•20 years ago
|
Flags: blocking1.8b2+ → blocking1.8b3+
Assignee | ||
Comment 7•20 years ago
|
||
I think I know what broke, but I don't know enough about the build system to fix
it. Here are the things that have changed since I orginally checked in everything:
new files in extensions/reporter/locales:
there is a new jar.mn that seems to be a template for localized files
there are en-US and generic directories with chrome files
changes in extensions/reporter/jar.mn:
bsmedberg's changes removed all the locale files from the reporter.jar, and it
looks like the new jar.mn file above should take care of those for different
locales, but i don't know how to make that work
in browser/installer/windows/reporter.jst
i removed the registerChrome line for the LOCALE, and the new files above
should take care of this as well, but i have no idea how
Those are the changes I think are causing problems in getting the reporter.xpi
created properly. How everything is linked and what to do to get the new
localized files included is something that I have not been able to figure out.
bsmedberg: I know you're busy, but if what I just noted makes sense to you, can
you either help me figure it out or take a couple of hours to look into it and
get it working? Any help is appreciated because both Robert and I are stumped
at this point. Thanks.
bryner: If you have some time, I can work with you too if you know what we need
to do here.
Comment 8•20 years ago
|
||
This is what I think you need, and it compiles on my tree. The change to
"iconv" and install.it are incidental l10n fixup I had hanging around in my
tree.
Attachment #182631 -
Attachment is obsolete: true
Attachment #183545 -
Flags: review?(robert)
Reporter | ||
Comment 9•20 years ago
|
||
Comment on attachment 183545 [details] [diff] [review]
Installer fixup
bryner is [much] more qualified to review this.
My box isn't building (cygwin can go to hell), and I don't get enough of this
stuff to intelegently look at it.
Attachment #183545 -
Flags: review?(robert) → review?(bryner)
Assignee | ||
Comment 10•20 years ago
|
||
Robert: I have a tree with all of the changes from your patch already in
(except for the iconv and the config.it changes)... So let me try adding those
couple of things later today and see if that works. I am getting builds right
now, but although the installer allows me to choose Reporter during a custom
install, I don't see it in the UI and the build log doesn't show the xpi being
built properly. If the two things that are missing fix it, I'll update this bug
with my patch and we can go from there.
Assignee | ||
Comment 11•20 years ago
|
||
I was on my way out when I left my last comment, but it looks like bsmedberg's
latest patch is what I already have in my tree..and it's not working! The two
changes I didn't have don't seem to be related to the problem (as he said
himself, that they were incidental l10n changes left over in his tree).
Soo...we're still back where we started. Any other ideas?
Comment 12•20 years ago
|
||
When I build an installer with this patch it works fine (the reporter appears in
the help menu, assuming I choose custom install +reporter -talkback). What is
the specific problem you're seeing?
Reporter | ||
Comment 13•20 years ago
|
||
If you can build and see it in the help menu... I'd say it's working.
Do we have an option in the installer?
Assignee | ||
Comment 14•20 years ago
|
||
I realized that I might still need the /reporter/resources/locale directory
(which I had deleted from my tree a while back after the l10n changes). I did a
"cvs up -d" from the extensions/reporter dir and am building now. I'll build
the installer in the morning and see what happens. Hopefully this was all my
mistake/stupidity and it works. :-) I'm learning as I go...
Assignee | ||
Comment 15•20 years ago
|
||
I have no idea what's wrong with my setup, it's still not working for me. I get
a -214: DOES NOT EXIST error during a custom install where I choose to install
Website Reporter.
I'm going to pull a fresh tree and try again, but if I still can't get it
working, I say we get the reviews we need and get these last few changes checked
in to see if it works for others (Since it appears to be working for Benjamin).
Reporter | ||
Comment 16•20 years ago
|
||
Agreed.
You can get the review(s) needed, checkin, and enable reporter at this point.
Once we have some successful builds, we'll point towards production, and life
will be good.
Assignee | ||
Updated•20 years ago
|
Attachment #183545 -
Flags: superreview?(bryner)
Attachment #183545 -
Flags: review?(jay)
Attachment #183545 -
Flags: review?(bryner)
Attachment #183545 -
Flags: approval1.8b2?
Attachment #183545 -
Flags: approval-aviary1.1a1?
Assignee | ||
Updated•20 years ago
|
Attachment #183545 -
Flags: superreview?(bryner)
Attachment #183545 -
Flags: review?(jay)
Attachment #183545 -
Flags: review?(chase)
Attachment #183545 -
Flags: approval1.8b2?
Attachment #183545 -
Flags: approval-aviary1.1a1?
Assignee | ||
Updated•20 years ago
|
Attachment #183545 -
Flags: superreview?(bryner)
Attachment #183545 -
Flags: approval1.8b2?
Attachment #183545 -
Flags: approval-aviary1.1a1?
Comment 17•20 years ago
|
||
Comment on attachment 183545 [details] [diff] [review]
Installer fixup
a=asa for landing as soon as reviews are in. Thanks, everyone, for all your
hard work on making this happen.
Attachment #183545 -
Flags: approval1.8b2?
Attachment #183545 -
Flags: approval1.8b2+
Attachment #183545 -
Flags: approval-aviary1.1a1?
Attachment #183545 -
Flags: approval-aviary1.1a1+
Updated•20 years ago
|
Attachment #183545 -
Flags: superreview?(bryner) → superreview+
Comment 18•20 years ago
|
||
Attachment #183545 -
Flags: review?(chase) → review+
Assignee | ||
Comment 19•20 years ago
|
||
Comment on attachment 183545 [details] [diff] [review]
Installer fixup
Checked in bsmedberg's suggested fix. I am still unable to get it working on
my machine, but he said it worked for him. Please test with tomorrow's builds
and see if Reporter is working.
Assignee | ||
Updated•20 years ago
|
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 20•20 years ago
|
||
For future reference, the problem was that I also needed to add reporter to
default list of extensions to be built in configure.in and run autoconf on my
local tree to get a build that reflected that change.
Status: RESOLVED → VERIFIED
Updated•6 years ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•