Bug 1095 - i18n team should be notified on string changes in svn
Summary: i18n team should be notified on string changes in svn
Status: RESOLVED FIXED
Alias: None
Product: Infrastructure
Classification: Unclassified
Component: Others (show other bugs)
Version: unspecified
Hardware: All Linux
Priority: High critical
Target Milestone: ---
Assignee: Sysadmin Team
QA Contact:
URL:
Whiteboard:
Keywords:
: 6377 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-05-02 12:17 CEST by Oliver Burger
Modified: 2014-11-05 16:55 CET (History)
10 users (show)

See Also:
Source RPM:
CVE:
Status comment:


Attachments

Description Oliver Burger 2011-05-02 12:17:38 CEST
Is it possible to auto generate mails to i18n, when there are string changes in the pot file.

It would make keeping resources in Tx up to date a lot easier.
Comment 1 Marek Laane 2011-05-02 12:24:39 CEST
It would be very welcome. At least KDE has some "magic" to do so automatically, see e.g. http://websvn.kde.org/trunk/kde-common/svn/hooks/post-commit.pl?view=markup

CC: (none) => bald

Comment 2 Nicolas Vigier 2011-05-02 12:28:25 CEST
I think there was something similar on mandriva svn, but can't find the script that was used.

CC: (none) => boklm

Comment 3 Marek Laane 2011-05-02 12:31:17 CEST
As far as I remember there was/is no such thing in Mandriva or at least it was/is not used, though there were times when Pixel or Thierry sent more or less regularly an email to the i18n list.
Comment 4 Nicolas Vigier 2011-05-02 12:35:41 CEST
The one mentioned in this thread :
http://lists.mandriva.com/cooker-i18n/2010-02/msg00023.php
Comment 5 Marek Laane 2011-05-02 13:08:30 CEST
I stand corrected - looked at Mandriva i18n archives http://lists.mandriva.com/cooker-i18n/ and remembered there really were such automatic emails in the first half of 2009. Have no idea why it was discontinued or where to look at relevant script.
Comment 6 Michael Scherer 2011-05-02 17:32:07 CEST
We could add a post-commit hook for that, but this would requires some change puppet side.

CC: (none) => misc

Comment 7 Michael Scherer 2011-05-02 17:36:27 CEST
Ok, I think I have a proposal that would not be too specific.

Assignee: sysadmin-bugs => misc

Comment 8 Michael Scherer 2011-05-02 17:46:37 CEST
I think it should be ok. But later, I think we should migrate to something more generic so people can subscribe freely to any change they want.

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

Comment 9 Michael Scherer 2011-05-03 16:56:31 CEST
It seems it didn't work, I have to debug this.

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

Comment 10 Michael Scherer 2011-07-20 23:11:56 CEST
Ok, I forgot about this, but I think I found the problem, I just send the mail to the wrong address. I committed a fix, and I will update the translation of identity for that. 

The mail is sent on mageia-i18n@
Comment 11 Michael Scherer 2011-07-21 02:46:42 CEST
After testing, it seems to not work. Further debugging have show that's because the perl module we use ( Svn::Notify::Config ) is only able to match on the path of a fle, not the filename itself. However, I was unable to remotely understand the code :/
Comment 12 Michael Scherer 2011-07-21 02:57:55 CEST
Ok, I had a stroke of genius, and found the code responsible, and this is in SVN::Notify, function  prepare_recipients. 

I guess replacing svnlook dirs-changed by svnlook changed + some filtering would do the trick.
Comment 13 Michael Scherer 2011-07-21 03:30:58 CEST
FYI : https://github.com/theory/svn-notify/pull/1
Comment 14 Oliver Burger 2011-10-03 10:03:26 CEST
Any news on this?
Comment 15 Marja Van Waes 2012-01-08 13:52:44 CET
pinging. because nothing happened to this report since more than 3 months ago, and it still has the status NEW or REOPENED


@ misc

Please set status to ASSIGNED if you think this bug was assigned correctly. If for work flow reasons you can't do that, then please put OK on the whiteboard instead.

CC: (none) => marja11

Manuel Hiebel 2012-03-22 17:09:58 CET

Priority: Normal => release_blocker

Oliver Burger 2012-03-26 22:10:14 CEST

Priority: release_blocker => High

Dimitrios Glentadakis 2012-04-19 12:20:01 CEST

CC: (none) => dglent

Filip Komar 2012-06-04 16:33:09 CEST

CC: (none) => filip.komar

Manuel Hiebel 2012-06-08 21:06:44 CEST

CC: (none) => mageia
Hardware: i586 => All
Assignee: misc => sysadmin-bugs

Comment 16 Manuel Hiebel 2012-06-08 21:06:56 CEST
*** Bug 6377 has been marked as a duplicate of this bug. ***

CC: (none) => thierry.vignaud

Comment 17 Thierry Vignaud 2012-06-11 19:40:34 CEST
This one _MUST_ be fixed before next release.
It would make developer & translator life quite a lot easier.

Priority: High => release_blocker
Severity: normal => critical

Romain d'Alverny 2012-06-26 10:27:12 CEST

CC: (none) => rdalverny

Comment 18 Oliver Burger 2012-07-18 10:04:50 CEST
Ping!
Comment 19 Anne Nicolas 2013-04-16 21:48:29 CEST
This is not release critical as it can be fixed at any time. But I do agree this should be fixed quickly. boklm will have a look as soon as possible

Priority: release_blocker => High
CC: (none) => ennael1

Nicolas Vigier 2014-03-24 10:46:17 CET

CC: boklm => (none)

Comment 20 Filip Komar 2014-11-05 16:55:57 CET
I guess we can close this one as Colin did his magic some time ago.

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


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