public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug other/23430] New: configure should allow --comment option
@ 2005-08-17  1:53 flash at pobox dot com
  2005-08-17  1:55 ` [Bug other/23430] " flash at pobox dot com
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: flash at pobox dot com @ 2005-08-17  1:53 UTC (permalink / raw)
  To: gcc-bugs

When trying to keep track of various builds of GCC, it would be convenient to be able to store a 
comment to be read back with -v.  Since -v echoes the full configure command line, all that's required 
is not rejecting an argument containing the desired comment.
    I'll upload a patch which permits this.  All it does is add a --comment option which does nothing; it's 
just seventeen copies of the following, one for each configure file:

  --comment=*)
     ;;                #Do nothing; this is just so that the comment will show up with -v. 

    There may be an easier way for maintainers to patch this, using auto-configuration; but I've never 
run that.  There are workarounds, such as embedding configuration information into directory names, 
but this approach seems simpler.
    A patched checking=yes build passed make check. 

Sample Session:
32> /opt2/gcc401-chk-all-4p/bin/g++ -v
Using built-in specs.
Target: i686-pc-linux-gnu
Configured with: /opt2/gcc-4.0.1-src/configure --enable-checking=all --prefix=/opt2/gcc401-chk-
all-4p/ --enable-languages=c,c++ --comment=PalmSource checking=all build by Flash Sheridan 
2005-08-11 via distcc.  Patches 20161-8277, 22508-2005-08-00326, 22600-9330, 
22604-2005-08-00334, plus a local patch to seventeen versions of configure to allow this comment.
Thread model: posix
gcc version 4.0.1

-- 
           Summary: configure should allow --comment option
           Product: gcc
           Version: 4.0.1
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: P2
         Component: other
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: flash at pobox dot com
                CC: gcc-bugs at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23430


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug other/23430] configure should allow --comment option
  2005-08-17  1:53 [Bug other/23430] New: configure should allow --comment option flash at pobox dot com
@ 2005-08-17  1:55 ` flash at pobox dot com
  2005-08-17  1:56 ` [Bug bootstrap/23430] " pinskia at gcc dot gnu dot org
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: flash at pobox dot com @ 2005-08-17  1:55 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From flash at pobox dot com  2005-08-17 01:53 -------
Created an attachment (id=9508)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=9508&action=view)
Proposed patch


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23430


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug bootstrap/23430] configure should allow --comment option
  2005-08-17  1:53 [Bug other/23430] New: configure should allow --comment option flash at pobox dot com
  2005-08-17  1:55 ` [Bug other/23430] " flash at pobox dot com
@ 2005-08-17  1:56 ` pinskia at gcc dot gnu dot org
  2005-08-17  9:39 ` schwab at suse dot de
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2005-08-17  1:56 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From pinskia at gcc dot gnu dot org  2005-08-17 01:56 -------
One thing you could do is replace the place where the configure options where, unless you want the 
configure options also.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
          Component|other                       |bootstrap
     Ever Confirmed|                            |1
   Last reconfirmed|0000-00-00 00:00:00         |2005-08-17 01:56:10
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23430


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug bootstrap/23430] configure should allow --comment option
  2005-08-17  1:53 [Bug other/23430] New: configure should allow --comment option flash at pobox dot com
  2005-08-17  1:55 ` [Bug other/23430] " flash at pobox dot com
  2005-08-17  1:56 ` [Bug bootstrap/23430] " pinskia at gcc dot gnu dot org
@ 2005-08-17  9:39 ` schwab at suse dot de
  2005-08-17 20:07 ` [Bug bootstrap/23430] --with-comment configure option isn't documented flash at pobox dot com
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: schwab at suse dot de @ 2005-08-17  9:39 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From schwab at suse dot de  2005-08-17 08:50 -------
Use --with-comment instead. 

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |INVALID


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23430


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug bootstrap/23430] --with-comment configure option isn't documented
  2005-08-17  1:53 [Bug other/23430] New: configure should allow --comment option flash at pobox dot com
                   ` (2 preceding siblings ...)
  2005-08-17  9:39 ` schwab at suse dot de
@ 2005-08-17 20:07 ` flash at pobox dot com
  2005-08-21 19:29 ` pinskia at gcc dot gnu dot org
  2005-08-21 20:22 ` pinskia at gcc dot gnu dot org
  5 siblings, 0 replies; 7+ messages in thread
From: flash at pobox dot com @ 2005-08-17 20:07 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From flash at pobox dot com  2005-08-17 19:41 -------
Thanks, --with-comment seems to work, but it doesn't seem to be documented.  Grepping both the 
documentation and source folder didn't find it; Googling for it turned it up in the context of MySQL, but 
not GCC.  So I'm re-titling and reopening the bug.  I think the option should be listed in --help, but it 
certainly needs to be documented somewhere.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|enhancement                 |minor
             Status|RESOLVED                    |REOPENED
         Resolution|INVALID                     |
            Summary|configure should allow --   |--with-comment configure
                   |comment option              |option isn't documented


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23430


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug bootstrap/23430] --with-comment configure option isn't documented
  2005-08-17  1:53 [Bug other/23430] New: configure should allow --comment option flash at pobox dot com
                   ` (3 preceding siblings ...)
  2005-08-17 20:07 ` [Bug bootstrap/23430] --with-comment configure option isn't documented flash at pobox dot com
@ 2005-08-21 19:29 ` pinskia at gcc dot gnu dot org
  2005-08-21 20:22 ` pinskia at gcc dot gnu dot org
  5 siblings, 0 replies; 7+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2005-08-21 19:29 UTC (permalink / raw)
  To: gcc-bugs



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|minor                       |enhancement
           Keywords|                            |documentation


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23430


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug bootstrap/23430] --with-comment configure option isn't documented
  2005-08-17  1:53 [Bug other/23430] New: configure should allow --comment option flash at pobox dot com
                   ` (4 preceding siblings ...)
  2005-08-21 19:29 ` pinskia at gcc dot gnu dot org
@ 2005-08-21 20:22 ` pinskia at gcc dot gnu dot org
  5 siblings, 0 replies; 7+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2005-08-21 20:22 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From pinskia at gcc dot gnu dot org  2005-08-21 19:29 -------
The main reason why it is documented, it is not a real configure option.   But we (autoconf) ignores 
unknown --with options.

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23430


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2005-08-21 19:29 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-08-17  1:53 [Bug other/23430] New: configure should allow --comment option flash at pobox dot com
2005-08-17  1:55 ` [Bug other/23430] " flash at pobox dot com
2005-08-17  1:56 ` [Bug bootstrap/23430] " pinskia at gcc dot gnu dot org
2005-08-17  9:39 ` schwab at suse dot de
2005-08-17 20:07 ` [Bug bootstrap/23430] --with-comment configure option isn't documented flash at pobox dot com
2005-08-21 19:29 ` pinskia at gcc dot gnu dot org
2005-08-21 20:22 ` pinskia at gcc dot gnu dot org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).