Bug 17093 - translate-shell doesn't work any more
Summary: translate-shell doesn't work any more
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 5
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL: https://github.com/soimort/translate-...
Whiteboard: has_procedure advisory MGA5-32-OK MGA...
Keywords: validated_update
Depends on:
Blocks:
 
Reported: 2015-11-06 06:47 CET by Dimitrios Glentadakis
Modified: 2015-11-17 09:08 CET (History)
3 users (show)

See Also:
Source RPM: translate-shell-0.9.0.8-1.mga5.src.rpm
CVE:
Status comment:


Attachments

Description Dimitrios Glentadakis 2015-11-06 06:47:31 CET
Our current version 0.9.0.8 doesn't work any more:
===
[dglent@localhost ~]$ trans fr:en maison
[ERROR] Null response.
[ERROR] Oops! Something went wrong and I can't translate it for you :(
===

The version 0.9.0.9 works correctly:
===
[dglent@localhost ~]$ trans fr:en maison
maison

House

Definitions maison
[ Français -> English ]

Noun
    house
        maison, chambre, famille, foyer, manoir, parlement
    home
        maison, domicile, foyer, pays, habitation, gîte
...
===



Reproducible: 

Steps to Reproduce:
Dimitrios Glentadakis 2015-11-06 06:56:27 CET

URL: (none) => https://github.com/soimort/translate-shell/issues/83

Comment 1 Dimitrios Glentadakis 2015-11-06 06:59:12 CET
Advisory:
========================

Updated translate-shell package makes the application again operational:

This application uses Google Translate API and for this,
every change of the API needs an update of the program.

References:
https://github.com/soimort/translate-shell/issues/83
https://bugs.mageia.org/show_bug.cgi?id=17093
========================

Updated packages in core/updates_testing:
========================
translate-shell-0.9.0.9-1mga5

Source RPMs: 
translate-shell-0.9.0.9-1mga5.src.rpm

Assignee: bugsquad => qa-bugs

Comment 2 Lewis Smith 2015-11-07 21:19:14 CET
Testing Mageia 5 x64.
Trouble is, unable to reproduce the fault:

BEFORE update: translate-shell-0.9.0.8-1.mga5
$ trans fr:en maison
maison

House

Definitions of maison
[ Français -> English ]
...

AFTER update: translate-shell-0.9.0.9-1.mga5
The entire output was identical (& correct) to previously.

BTAIM There is no man entry for trans. Try:
 $ trans -h           for overall help
 $ trans -T           for the language codes
The help output ends: "See the man page trans(1) for more information."
So, should there be one?

OKing on the basis of no reversion, and it seems to work as expected.

CC: (none) => lewyssmith
Whiteboard: (none) => MGA5-64-OK

Comment 3 Dimitrios Glentadakis 2015-11-08 06:40:59 CET
Nice catch Lewis.
I fixed the man path and uploaded a new rpm.
Now is possible to view the man page with:
<man trans>
or in konqueror with:
<man:trans>

The bug is reproductible for a range of IP.
See: https://github.com/soimort/translate-shell/issues/83#issuecomment-145354367
So it seems that you are not affected.
Comment 4 Dimitrios Glentadakis 2015-11-08 06:44:00 CET
Advisory:
========================

Updated translate-shell package fixes a problem with a block of a certain range of IPs by Google Translate API.

References:
https://github.com/soimort/translate-shell/issues/83
https://bugs.mageia.org/show_bug.cgi?id=17093
========================

Updated packages in core/updates_testing:
========================
translate-shell-0.9.0.9-2mga5

Source RPMs: 
translate-shell-0.9.0.9-2mga5.src.rpm
Comment 5 Dimitrios Glentadakis 2015-11-08 06:57:31 CET
Advisory:
========================

Updated translate-shell package fixes a problem with a block of a certain range of IPs by Google Translate API.
It fixes also the path of the man page to make it available from the man command.

References:
https://github.com/soimort/translate-shell/issues/83
https://bugs.mageia.org/show_bug.cgi?id=17093
========================

Updated packages in core/updates_testing:
========================
translate-shell-0.9.0.9-2mga5

Source RPMs: 
translate-shell-0.9.0.9-2mga5.src.rpm
Comment 6 Lewis Smith 2015-11-08 21:32:28 CET
Thanks Dimitrios. Yes, your reference in Comment 3 does say that the failure was Internet address related.
I wonder whether the Comment 5 Advisory should indicate that the man page is for the 'trans' command. How would one know from the package name?

Testing again M5 x64

BEFORE 2nd update: translate-shell-0.9.0.9-1.mga5
 $ man trans
 No manual entry for trans

AFTER 2nd update: translate-shell-0.9.0.9-2.mga5
 $ man trans      [man page displays]

 $ trans fr:en maison
 maison
 House
 Definitions of maison
 [ Français -> English ]
 noun
 ...
And it still works. So all looks OK, and the update OK is better.
Comment 7 Dimitrios Glentadakis 2015-11-09 02:13:26 CET
I think you have right for the trans command, and not (only) in the advisory but in the package description we could add eg: "type <man trans> for more information".
WDYT ?
Comment 8 Lewis Smith 2015-11-09 19:14:01 CET
(In reply to Dimitrios Glentadakis from comment #7)
> I think you have right for the trans command, and not (only) in the advisory
> but in the package description we could add eg: "type <man trans> for more
> information".
> WDYT ?
Certainly. Anything which clarifies its usage.
Comment 9 Len Lawrence 2015-11-10 20:04:33 CET
@Lewis
Are you OK for testing this on i586?  If not I can do it in a vbox.

CC: (none) => tarazed25

Comment 10 Dimitrios Glentadakis 2015-11-11 07:31:11 CET
(In reply to Lewis Smith from comment #8)
> (In reply to Dimitrios Glentadakis from comment #7)
> > I think you have right for the trans command, and not (only) in the advisory
> > but in the package description we could add eg: "type <man trans> for more
> > information".
> > WDYT ?
> Certainly. Anything which clarifies its usage.

Ok in Cauldron:
http://svnweb.mageia.org/packages/cauldron/translate-shell/current/SPECS/translate-shell.spec?r1=902014&r2=902013&pathrev=902014
Comment 11 Dimitrios Glentadakis 2015-11-11 07:32:12 CET
Advisory:
========================
 
Updated translate-shell package fixes a problem with a block of a certain
range of IPs by Google Translate API.
It fixes also the path of the man page to make it available from the man
command.
Run <man trans> in the terminal for usage information.

References:
https://github.com/soimort/translate-shell/issues/83
https://bugs.mageia.org/show_bug.cgi?id=17093
========================

Updated packages in core/updates_testing:
========================
translate-shell-0.9.0.9-2mga5
 
Source RPMs: 
translate-shell-0.9.0.9-2mga5.src.rpm
Comment 12 Lewis Smith 2015-11-12 20:45:16 CET
(In reply to Len Lawrence from comment #9)
> Are you OK for testing this on i586?  If not I can do it in a vbox.
Thanks for popping up. I have *only* x64, so cannot do 32-bit. With the notes in place, this update should be quick to try.
Comment 13 Len Lawrence 2015-11-12 21:06:55 CET
Thanks Lewis.  Will try to get this done before it gets pushed.
Comment 14 Len Lawrence 2015-11-12 21:16:43 CET
Installed translate-shell in i586 virtualbox
4.1.12-desktop-1.mga5

[lcl@cursa ~]$ trans fr:en battaille
bataille

battle

Definitions of bataille
[ Français -> English ]

noun
    battle
        bataille, combat, lutte, guerre
    fight
        lutte, combat, bataille, bagarre, dispute, pugilat
    combat
        combat, bataille, conflit
    sword
        épée, sabre, glaive, lame, bataille, duel à l'épée

[lcl@cursa ~]$ trans en:it whatever
whatever
/(h)wÉtËevÉr,Ë(h)wät-/

che cosa mai

Definitions of whatever
[ English -> Italiano ]

adjective
    qualsiasi
        any, all, whatever, whichever, whatsoever
    qualunque
        any, whatever, whichever, ordinary
    alcuno
        any, some, whatever, all

pronoun
    qualunque cosa
        whatever, whichever
    qualsiasi cosa
        whatever
    ciò che
        what, whatever
    tutto ciò che
        whatever
    quello che
        what, whatever

Comprehensive or what!
Len Lawrence 2015-11-12 21:17:01 CET

Whiteboard: MGA5-64-OK => MGA5-32-OK MGA5-64-OK

Comment 15 Rémi Verschelde 2015-11-16 08:32:32 CET
Don't hesitate to put the "validated_update" keyword yourself Len when both arches have been successfully tested.

Keywords: (none) => validated_update
CC: (none) => sysadmin-bugs

Comment 16 claire robinson 2015-11-16 17:40:08 CET
Advisory added, corrected srpm to translate-shell-0.9.0.9-2.mga5

Whiteboard: MGA5-32-OK MGA5-64-OK => has_procedure advisory MGA5-32-OK MGA5-64-OK

Comment 17 Mageia Robot 2015-11-16 22:37:37 CET
An update for this issue has been pushed to Mageia Updates repository.

http://advisories.mageia.org/MGAA-2015-0182.html

Status: NEW => RESOLVED
Resolution: (none) => FIXED

Comment 18 Len Lawrence 2015-11-17 09:08:51 CET
@Rémi: Yes, thanks.  I am always a bit hesitant with such things.

Note You need to log in before you can comment on or make changes to this bug.