Modify

Ticket #237 (closed defect: invalid)

Opened 6 years ago

Last modified 4 years ago

moap should detect an email address from the environment that is already enclosed by < and >

Reported by: zaheermerali@… Owned by: thomas
Priority: major Milestone:
Component: moap Version: TRUNK
Keywords: Cc:

Description

moap should detect an email address from the environment that is already enclosed by < and >

Attachments

Change History

comment:1 Changed 6 years ago by anonymous

Could you give an example ? I don't understand the ticket.

comment:2 Changed 6 years ago by zaheermerali@…

Sorry lacks context. If my EMAIL_ADDRESS environment variable already contains my email address enclosed inside < >, moap should not add an extra < and > when preparing ChangeLog?.

comment:3 Changed 6 years ago by thomas

I've never seen anyone use EMAIL_ADDRESS with angle brackets in it. Why do you have that ?

comment:4 Changed 6 years ago by zaheermerali@…

Was a mistake, but probably very easy to handle case for moap cl prepare. I believe cl.pl handled it ok.

comment:5 Changed 6 years ago by thomas

from my copy of cl.pl:

my $email_address = $ENV{CHANGE_LOG_EMAIL_ADDRESS}
  || $ENV{EMAIL_ADDRESS}
  || "set EMAIL_ADDRESS environment variable";
open CHANGE_LOG, "> ChangeLog" or die "Could not write ChangeLog\n.";
print CHANGE_LOG "$date  $name  <$email_address>\n\n";

It doesn't look like it does what you say.

I should add CHANGE_LOG_EMAIL_ADDRESS support though :)

comment:6 Changed 6 years ago by zaheermerali@…

aaah that explains it my CHANGE_LOG_EMAIL_ADDRESS had it without the angle brackets but my EMAIL_ADDRESS had it with!

comment:7 Changed 6 years ago by thomas

see [220] for the CHANGE_LOG_EMAIL_ADDRESS change

comment:8 Changed 6 years ago by thomas

  • Status changed from new to closed
  • Resolution set to invalid
View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.