perl-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stas Bekman <s...@stason.org>
Subject Re: cvs commit: modperl-2.0/xs/tables/current/Apache ConstantsTable.pm
Date Thu, 12 Aug 2004 05:03:58 GMT
Geoffrey Young wrote:
>>Have you by chance noticed:
>>
>>% cat xs/tables/current/Apache/ConstantsTable.pm
>>package Apache::ConstantsTable;
>>
>># !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
>># ! WARNING: generated by Apache::ParseSource/0.02
>># !          Mon May  3 13:30:11 2004
>># !          do NOT edit, any changes will be lost !
>># !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
>>
>>The next time the source scan will be run, your changes will be lost.
> 
> 
> in all fairness, we've hand-edited that file tons of times, so it's not
> really like there is no precident for it (and I'm the one who told him to
> just add them :)

Luckily, that is not the case for a few months already. And it saved me 
a lot of time, since I started working on the API polish.

> I guess that along with the edit is adding the 'context' group to
> Apache::ParseSource so that the table is correctly generated next time
> somebody runs it.

No, not really. 'make source_scan' will do the edit for you. That's the 
whole point of it. The only reason we were doing it manually is because 
it was broken. It works now. So one needs to tweaks ParseSource to pick 
up that group.

> sorry.

No problem :)


-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org


Mime
View raw message