Closed
Bug 662580
Opened 15 years ago
Closed 15 years ago
makefiles: add debug macros and 'help' documentation targets
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: joey, Assigned: joey)
Details
Attachments
(1 file, 1 obsolete file)
|
12.85 KB,
patch
|
Details | Diff | Splinter Review |
Add help targets to begin documenting makefile elements.
Add trace macros [triggered on demand] to aide in debugging
| Assignee | ||
Updated•15 years ago
|
Assignee: nobody → joey
| Assignee | ||
Updated•15 years ago
|
Attachment #537831 -
Flags: review?(coop)
| Assignee | ||
Comment 1•15 years ago
|
||
Comment on attachment 537831 [details] [diff] [review]
Make debug macros and a skeleton help hierarchy
non-invasive help targets that can be used to document makefile elements
Attachment #537831 -
Flags: review?(khuey)
Attachment #537831 -
Flags: review?(coop)
Attachment #537831 -
Flags: review?(benjamin)
Updated•15 years ago
|
Attachment #537831 -
Flags: review?(khuey)
Comment 2•15 years ago
|
||
Comment on attachment 537831 [details] [diff] [review]
Make debug macros and a skeleton help hierarchy
Is this supposed to be documentation of the meaning of makefile variables, or their values? We already have rules to dump the values of the most important variables at the end of rules.mk. But in general, I think it's kinda weird/overkill to use makefiles to document the system, instead of a README file or our standard doc website. We have previously tried to document the various variable meanings on MDC, and I think I'd prefer to keep it there instead of moving all that in-tree. Then "make help" could just be
help:
echo "See https://developer.mozilla.org/en/How_Mozilla%27s_build_system_works for details about the Mozilla build system."
Also, it seems really complex to use make to provide these docs, instead of writing a README.build file.
Ted should probably make the final decision on this, though, since he's the module owner.
Attachment #537831 -
Flags: review?(ted.mielczarek)
Attachment #537831 -
Flags: review?(khuey)
Attachment #537831 -
Flags: review?(benjamin)
| Assignee | ||
Comment 3•15 years ago
|
||
Same basic set of files included with the earlier patch.
Removed a few stray files that were checked in.
Renamed a few files to match the *.mk convention.
Attachment #537831 -
Attachment is obsolete: true
Attachment #537831 -
Flags: review?(ted.mielczarek)
Attachment #538930 -
Flags: review?(khuey)
Attachment #538930 -
Flags: review?(benjamin)
| Assignee | ||
Comment 4•15 years ago
|
||
(In reply to comment #2)
> Comment on attachment 537831 [details] [diff] [review] [review]
> Make debug macros and a skeleton help hierarchy
>
> Is this supposed to be documentation of the meaning of makefile variables,
> or their values? We already have rules to dump the values of the most
> important variables at the end of rules.mk. But in general, I think it's
> kinda weird/overkill to use makefiles to document the system, instead of a
> README file or our standard doc website. We have previously tried to
> document the various variable meanings on MDC, and I think I'd prefer to
> keep it there instead of moving all that in-tree. Then "make help" could
> just be
> help:
> echo "See
> https://developer.mozilla.org/en/How_Mozilla%27s_build_system_works for
> details about the Mozilla build system."
>
> Also, it seems really complex to use make to provide these docs, instead of
> writing a README.build file.
>
> Ted should probably make the final decision on this, though, since he's the
> module owner.
Ok I'll check on the wiki writeup.
The intent was to provide help directly within the sandbox w/o having to search for content offline. Grep can be used for specifics:
% cd obj--ff-dbg/db
% gmake help.variables | grep FOOBAR
The structure would also allow documenting macro values { possibly derived }:
[phantasm:db] gmake help.variables.OS_ARCH
===========================================================================
Topic: help.variables.OS_ARCH
===========================================================================
AIX
ANDROID
BSD_OS
BeOS
DGUX
Darwin
DragonFly
FreeBSD
| Assignee | ||
Updated•15 years ago
|
Attachment #538930 -
Flags: review?(ted.mielczarek)
Attachment #538930 -
Flags: review?(khuey)
Attachment #538930 -
Flags: review?(benjamin)
| Assignee | ||
Comment 5•15 years ago
|
||
Preferred answer is to maintain text in MDC/wiki rather than generating content from the sandbox. This will allow the community to be involved with maintaining documentation if they choose to.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Updated•15 years ago
|
Resolution: FIXED → WONTFIX
Updated•15 years ago
|
Attachment #538930 -
Flags: review?(ted.mielczarek)
Updated•8 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•