#10004: Enable -c switch in xgettext call to collect translator comments
-------------------------------------------+--------------------------------
          Reporter:  claudep               |         Owner:              
            Status:  new                   |     Milestone:  1.3         
         Component:  Internationalization  |       Version:  1.0         
        Resolution:                        |      Keywords:  makemessages
             Stage:  Accepted              |     Has_patch:  1           
        Needs_docs:  1                     |   Needs_tests:  0           
Needs_better_patch:  0                     |  
-------------------------------------------+--------------------------------
Changes (by claudep):

  * needs_docs:  0 => 1
  * has_patch:  0 => 1

Comment:

 When reviewing martinb's patch, I wonder what value we add in making the
 addcomments switch optional and the extraction keyword configurable.

 I have proposed a new patch, much simpler, which simply extract comments
 with the L10n keyword. Extracting translator comments should never be
 optional IMHO.

 About tests, it's slightly messed up in the patch because I reorganized
 them to not require a new class for each type of test. The only real new
 test is the test_comments_extractor method.

-- 
Ticket URL: <http://code.djangoproject.com/ticket/10004#comment:12>
Django <http://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Reply via email to