Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [RFA] Fix my email addresses
@ 2009-05-14 21:01 Pierre Muller
  2009-05-15  6:58 ` Eli Zaretskii
  0 siblings, 1 reply; 4+ messages in thread
From: Pierre Muller @ 2009-05-14 21:01 UTC (permalink / raw)
  To: gdb-patches

  I started using mklog.pl
a while ago, but didn't know that
the at character needed to be escaped in the file.
  As a result mu email address appears scrambled
in several ChangeLog entries :(

Sorry about that!

  I wrote a patch and added a fix to MAINTAINERS
renamed my address from
  muller@sources.redhat.com
to
  muller@sourceware.org


Is this OK?


Should I really add this entry to ChangeLog,
or commit it without ChangeLog entry?



Pierre Muller
Pascal language support maintainer for GDB



2009-05-14  Pierre Muller  <muller@ics.u-strasbg.fr>

	* ChangeLog: Fix wrong email addresses of myself.
	* MAINTAINERS: Update my email address.
	


Index: ChangeLog
===================================================================
RCS file: /cvs/src/src/gdb/ChangeLog,v
retrieving revision 1.10470
diff -u -r1.10470 ChangeLog
--- ChangeLog	14 May 2009 19:16:28 -0000	1.10470
+++ ChangeLog	14 May 2009 20:53:20 -0000
@@ -4,7 +4,7 @@
 	gdbarch.h does NOT need to be regenerated as the corresponding
 	change has already been made in a previous commit.
 
-2009-05-14  Pierre Muller  <muller.u-strasbg.fr>
+2009-05-14  Pierre Muller  <muller@ics.u-strasbg.fr>
 
 	Remove all i386 debug register low level macros in config nm files.
 	(I386_DR_LOW_SET_ADDR, I386_DR_LOW_RESET_ADDR): Remove.
@@ -167,7 +167,7 @@
 	(lin_thread_get_thread_signals): blocked_mask is global again.
 	Adjust.
 
-2009-05-11  Pierre Muller  <muller.u-strasbg.fr>
+2009-05-11  Pierre Muller  <muller@ics.u-strasbg.fr>
 
 	Unify target macros.
 
@@ -198,7 +198,7 @@
 	(debug_to_stopped_by_watchpoint): Likewise.
 
 
-2009-05-10  Pierre Muller  <muller.u-strasbg.fr>
+2009-05-10  Pierre Muller  <muller@ics.u-strasbg.fr>
 
 	* src/gdb/target.h: Remove all tests for already defined
 	macros. All macros defined here should not be set in config
@@ -252,7 +252,7 @@
 	call to ...
 	(add_inferior_silent): ... here.
 
-2009-05-06  Pierre Muller  <muller.u-strasbg.fr>
+2009-05-06  Pierre Muller  <muller@ics.u-strasbg.fr>
 	    Pedro Alves  <pedro@codesourcery.com>
 
 	* procfs.c (procfs_insert_watchpoint, procfs_remove_watchpoint)
@@ -539,7 +539,7 @@
 
 	* stack.c (_initialize_stack) <disassemble-next-line>: Doc fix.
 
-2009-04-24  Pierre Muller  <muller.u-strasbg.fr>
+2009-04-24  Pierre Muller  <muller@ics.u-strasbg.fr>
 
 	ARI change: Use "/* ARI: rule */" pattern.
 	* utils.c: Update ARI patterns.
@@ -645,7 +645,7 @@
 	* configure.ac: Use AC_HELP_STRING.  Use upper case for variables
 	in help.
 
-2009-04-21  Pierre Muller  <muller.u-strasbg.fr>
+2009-04-21  Pierre Muller  <muller@ics.u-strasbg.fr>
 
 	ARI fix: remove "%p".
 	* ia64-tdep.c (ia64_frame_this_id): Use host_address_to_string
function.
@@ -663,7 +663,7 @@
 	* main.c: Include "source.h".
 	(captured_main): Add substitution rule if RELOC_SRCDIR.
 
-2009-04-21  Pierre Muller  <muller.u-strasbg.fr>
+2009-04-21  Pierre Muller  <muller@ics.u-strasbg.fr>
 
 	* gnu-nat.h (proc_debug): Add missing continuation line in macro.
 
@@ -748,7 +748,7 @@
 	configure substitutions.
 	(FLAGS_TO_PASS): Add datarootdir, docdir, and htmldir.
 
-2009-04-17  Pierre Muller  <muller.u-strasbg.fr>
+2009-04-17  Pierre Muller  <muller@ics.u-strasbg.fr>
 
 	ARI fix: Do not use %p, replace by call to host_address_to_string
 	for host pointers.
Index: MAINTAINERS
===================================================================
RCS file: /cvs/src/src/gdb/MAINTAINERS,v
retrieving revision 1.417
diff -u -r1.417 MAINTAINERS
--- MAINTAINERS	14 May 2009 08:54:09 -0000	1.417
+++ MAINTAINERS	14 May 2009 20:53:20 -0000
@@ -442,7 +442,7 @@
 stabs reader		Elena Zannoni		elena.zannoni@oracle.com
 readline/		Elena Zannoni		elena.zannoni@oracle.com
 NetBSD native & host	Jason Thorpe		thorpej@netbsd.org
-Pascal support		Pierre Muller		muller@sources.redhat.com
+Pascal support		Pierre Muller		muller@sourceware.org
 avr			Theodore A. Roth	troth@openavr.org
 Modula-2 support	Gaius Mulley		gaius@glam.ac.uk
 
@@ -543,7 +543,7 @@
 Alan Modra					amodra@bigpond.net.au
 Jason Molenda					jmolenda@apple.com
 Phil Muldoon					pmuldoon@redhat.com
-Pierre Muller					muller@sources.redhat.com
+Pierre Muller					muller@sourceware.org
 Gaius Mulley                                    gaius@glam.ac.uk
 Joseph Myers					joseph@codesourcery.com
 Fernando Nasser					fnasser@redhat.com


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

* Re: [RFA] Fix my email addresses
  2009-05-14 21:01 [RFA] Fix my email addresses Pierre Muller
@ 2009-05-15  6:58 ` Eli Zaretskii
  2009-05-15  9:29   ` Joel Brobecker
  2009-05-15 10:15   ` Pierre Muller
  0 siblings, 2 replies; 4+ messages in thread
From: Eli Zaretskii @ 2009-05-15  6:58 UTC (permalink / raw)
  To: Pierre Muller; +Cc: gdb-patches

> From: "Pierre Muller" <muller@ics.u-strasbg.fr>
> Date: Thu, 14 May 2009 23:01:45 +0200
> 
>   I started using mklog.pl
> a while ago, but didn't know that
> the at character needed to be escaped in the file.
>   As a result mu email address appears scrambled
> in several ChangeLog entries :(
> 
> Sorry about that!
> 
>   I wrote a patch and added a fix to MAINTAINERS
> renamed my address from
>   muller@sources.redhat.com
> to
>   muller@sourceware.org
> 
> 
> Is this OK?

Yes, okay.

> Should I really add this entry to ChangeLog,
> or commit it without ChangeLog entry?

Personally, I think that such changes don't need a ChangeLog entry,
but since my opinions about ChangeLog's are more often than not
overruled by the majority of other developers, please wait for someone
else to speak up.

Thanks.

> 	* MAINTAINERS: Update my email address.

This entry does need to be in ChangeLog, IMO.


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

* Re: [RFA] Fix my email addresses
  2009-05-15  6:58 ` Eli Zaretskii
@ 2009-05-15  9:29   ` Joel Brobecker
  2009-05-15 10:15   ` Pierre Muller
  1 sibling, 0 replies; 4+ messages in thread
From: Joel Brobecker @ 2009-05-15  9:29 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Pierre Muller, gdb-patches

> > Should I really add this entry to ChangeLog,
> > or commit it without ChangeLog entry?
> 
> Personally, I think that such changes don't need a ChangeLog entry,
> but since my opinions about ChangeLog's are more often than not
> overruled by the majority of other developers, please wait for someone
> else to speak up.

I agree with Eli, changes to the ChangeLog don't need an associated
ChangeLog entry.

-- 
Joel


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

* RE: [RFA] Fix my email addresses
  2009-05-15  6:58 ` Eli Zaretskii
  2009-05-15  9:29   ` Joel Brobecker
@ 2009-05-15 10:15   ` Pierre Muller
  1 sibling, 0 replies; 4+ messages in thread
From: Pierre Muller @ 2009-05-15 10:15 UTC (permalink / raw)
  To: 'Eli Zaretskii', 'Joel Brobecker'; +Cc: gdb-patches

From Eli Zaretskii
> > Is this OK?
> 
> Yes, okay.

Thanks.
 
> > Should I really add this entry to ChangeLog,
> > or commit it without ChangeLog entry?
> 
> Personally, I think that such changes don't need a ChangeLog entry,
> but since my opinions about ChangeLog's are more often than not
> overruled by the majority of other developers, please wait for someone
> else to speak up.

  As Joel agreed with you,
I committed the ChangeLog fix first,
without adding anything on top,
and committed the MAINTAINERS file patch,
this time with a ChangeLog entry.

Pierre


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

end of thread, other threads:[~2009-05-15 10:15 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-05-14 21:01 [RFA] Fix my email addresses Pierre Muller
2009-05-15  6:58 ` Eli Zaretskii
2009-05-15  9:29   ` Joel Brobecker
2009-05-15 10:15   ` Pierre Muller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox