[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