Ticket #4083 enhancement closed fixed

Opened 4 years ago

Last modified 3 years ago

tap2rpm's default long description should mention tap2rpm, not tap2deb

Reported by: TimAllen Owned by:
Priority: normal Milestone:
Component: core Keywords: easy
Cc: thijs Branch:
Author: Launchpad Bug:

Description

Currently, twisted.scripts.tap2rpm contains the following line of code:

long_description = (config['long_description']
		    or 'Automatically created by tap2deb')

It should of course say "Automatically created by tap2rpm".

Attachments

4083.patch Download (2.6 KB) - added by Poincare 4 years ago.
reivew
4083.2.patch Download (2.6 KB) - added by Poincare 4 years ago.
Please reivew

Change History

1

Changed 4 years ago by TimAllen

This ticket split off from #3292.

Changed 4 years ago by Poincare

reivew

Changed 4 years ago by Poincare

Please reivew

2

Changed 3 years ago by jonathanj

  • keywords review added
  • owner glyph deleted

3

Changed 3 years ago by jonathanj

  • owner set to Poincare
  • keywords review removed

Perhaps I'm confused, but this ticket talks about tap2rpm's description while the patch also deletes aimbot.py. What's up with that?

4

Changed 3 years ago by thijs

  • cc thijs added

5

Changed 3 years ago by thijs

  • owner changed from Poincare to thijs
  • status changed from new to assigned

I'll create a new patch and apply it.

6

Changed 3 years ago by thijs

New patch:

  • twisted/scripts/tap2rpm.py

     
    225225    description = config['description'] or ('A TCP server for %(protocol)s' % 
    226226                                            vars()) 
    227227    long_description = (config['long_description'] 
    228                         or 'Automatically created by tap2deb') 
     228                        or "Automatically created by tap2rpm") 
    229229    twistd_option = type_dict[config['type']] 
    230230    date = time.strftime('%a %b %d %Y', time.localtime(time.time())) 
    231231    directory = rpm_file + '-' + version 

7

Changed 3 years ago by thijs

  • status changed from assigned to closed
  • resolution set to fixed

(In [27636]) Fix the default long description of tap2rpm, that incorrectly was talking about tap2deb.

Author: Poincare Reviewer: jonathanj, thijs Fixes: #4083

8

Changed 2 years ago by <automation>

  • owner thijs deleted
Note: See TracTickets for help on using tickets.