[Bug 453465] Review Request: transbot - An irc bot designed to translate between languages on separate channels

bugzilla at redhat.com bugzilla at redhat.com
Tue Aug 5 19:13:15 UTC 2008


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=453465





--- Comment #10 from John McLean <jesusfreak91 at gmail.com>  2008-08-05 15:13:14 EDT ---
(In reply to comment #9)
> Well,
> 
> * Currently I don't know how to use this, however when I just type $ transbot,
>   I get a traceback as
> ---------------------------------------------------------------------------
> $ transbot 
> Traceback (most recent call last):
>   File "/usr/bin/transbot", line 37, in <module>
>     channel_list = conf_file.readline().split ( ':' ).lower() [ 1 ]
> AttributeError: 'list' object has no attribute 'lower'
> ----------------------------------------------------------------------------
>    Is this expected?
> 
The traceback came from a typo.  I'll need to go back and fix that. 

> (In reply to comment #7)
> > -The latest source tarball has been uploaded at the same url.  I simply forgot
> > to upload it last time I made changes.  I'm still forming habits concerning
> > uploading changes and bumping the release numbers.
> 
> * What I asked you is to change the _version_ number when you modify the
> tarball
>   itself (release number is rpm specific). 
> 
>   Changing tarball without changing version may confuse people who are trying
> to 
>   use this package.

Alright.  I guess I read your comment too quickly and equated version with
release in my mind.  My apologies.  For future reference, should the version
number be changed even for one-line corrections?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the Fedora-package-review mailing list