Closed
Bug 140034
Opened 23 years ago
Closed 22 years ago
RFE: about:buildinfo -- displays 'build options' used
Categories
(SeaMonkey :: Build Config, enhancement, P4)
SeaMonkey
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.4alpha
People
(Reporter: arthit.suriyawongkul, Assigned: netscape)
References
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
about:buildinfo
(type this in the address bar)
displays
- 'build options' that has been used at the time of building the current running
software
- 'build number' (same as the title bar)
- date/time that it has been built
- other related info
these will help a lot for testing :)
Comment 1•23 years ago
|
||
To build config.
Assignee: Matti → seawood
Component: Browser-General → Build Config
QA Contact: imajes-qa → granrose
Assignee | ||
Comment 2•23 years ago
|
||
Yep, it would be nice though I'm not sure if we need to bloat the app with that
info. Regardless, I have no idea how to plug this into the about: handler so
it's going to be put off for a bit.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P4
Target Milestone: --- → mozilla1.2alpha
Comment 3•23 years ago
|
||
Belongs in "about:", surely. As fine print under the current listing of the
User-Agent.
It can be put in as 'fine print' that the nontechnical user's eyes will fall off
and ignore ;-)
Assignee | ||
Updated•22 years ago
|
Target Milestone: mozilla1.2alpha → Future
Assignee | ||
Comment 4•22 years ago
|
||
This should cover the basics.
Assignee | ||
Updated•22 years ago
|
Target Milestone: Future → mozilla1.4alpha
Assignee | ||
Updated•22 years ago
|
Attachment #113328 -
Flags: review?(bbaetz)
Comment 5•22 years ago
|
||
Comment on attachment 113328 [details] [diff] [review]
v1.0
+ CC_VERSION=`$CC --version`
That gives me:
gcc (GCC) 3.2 20020903 (Red Hat Linux 8.0 3.2-7)
Copyright (C) 2002 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
which is a bit long.. We probably want -dumpversion instead, or pass it through
sed, or soemthing. I dont' know how far -dumpversion goes back. If you do that,
then you need to add 'GCC' in there somewhere
What if that option doesn't exist? Won't configure exist with an error codeif
teh backticks fail?
More importantly, you need to html-escape the stuff in the html file. I also
don't see somewhere which triggers the .in file to be seen by configure. It is
possible for --version to have < or > in it.
The HTML should also become valid, have a doctype added, and so on.
Attachment #113328 -
Flags: review?(bbaetz) → review-
Assignee | ||
Comment 6•22 years ago
|
||
I tested with gcc 2.95.2 which just returns 2.95.2 for --version. 2.96 just
returns 2.96 as well.
--version is only used inside the $GCC = "yes" & $GXX = "yes" blocks so it
wouldn't fail unless gcc stopped supporting it. Otherwise, CC_VERSION &
CXX_VERSION default to 'N/A'.
buildconfig.html is listed in allmakefiles.sh. That's how configure.in sees it
and knows to generate it. What needs to be escaped? I don't plan to have all
of that extra copyright info from the --version output. At most, it will be the
first line which contains the binary name (GCC) version number and maybe the
vendor variant.
Bleh. What's "invalid" about the HTML other than the missing doctype declaration?
Comment 7•22 years ago
|
||
I missed the allmakefiles.sh change
That block of text is what gc-3.2 prints out for --version, though, so it needs
to be adjusted. I have this vague recollection of that being an annoucnced
change somewhere, but ICBW.
You have whole lot of <P> tags, with no </P>, or any content between that and
the beginning of the following <hN>
Assignee | ||
Comment 8•22 years ago
|
||
Use '$CC -v' output containing 'gcc version' instead of --version.
Let composer generate buildconfig.html.in so html should be valid.
Assignee | ||
Updated•22 years ago
|
Attachment #113328 -
Attachment is obsolete: true
Comment 9•22 years ago
|
||
Comment on attachment 113873 [details] [diff] [review]
v1.1
r=bbaetz if you change the <td><strong> stuff to use <th> instead.
Attachment #113873 -
Flags: review+
Assignee | ||
Comment 10•22 years ago
|
||
* Changed <td><strong> to <th>
Attachment #113873 -
Attachment is obsolete: true
Assignee | ||
Updated•22 years ago
|
Attachment #113879 -
Flags: superreview?(darin)
Comment 11•22 years ago
|
||
Comment on attachment 113879 [details] [diff] [review]
v1.2
looks fine to me (sr=darin)
Attachment #113879 -
Flags: superreview?(darin) → superreview+
Assignee | ||
Comment 12•22 years ago
|
||
Patch has been checked in.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•