06-12-2019, 05:49 PM,
|
|
falkoner
Tim Schofield
   
|
Posts: 1,215
Threads: 17
Joined: Mar 2015
|
|
RE: Pending 4.15.1 release
Hi Paul, I am pretty sure that the file names, and line numbers are for reference purposes only and do not get compiled into the .mo files.
It was the line wrap stuff that was concerning me, as I think that was what Rafael was striving to stop. It seems you have it sorted
Tim
|
|
06-13-2019, 08:59 PM,
|
|
falkoner
Tim Schofield
   
|
Posts: 1,215
Threads: 17
Joined: Mar 2015
|
|
RE: Pending 4.15.1 release
(06-12-2019, 05:49 PM)falkoner Wrote: Hi Paul, I am pretty sure that the file names, and line numbers are for reference purposes only and do not get compiled into the .mo files.
It was the line wrap stuff that was concerning me, as I think that was what Rafael was striving to stop. It seems you have it sorted 
Tim
Paul, I notice that Rafael has committed changes to the translation files, but to the master branch rather than the release branch. This concerns me because:
1 The new translation strings aren't in the release, and
2 This will cause you a big headache when you come to merge the release branch back into master.
Tim
|
|
06-15-2019, 01:05 PM,
(This post was last modified: 06-15-2019, 01:09 PM by TurboPT.)
|
|
TurboPT
Senior Member
   
|
Posts: 617
Threads: 7
Joined: Jun 2012
|
|
RE: Pending 4.15.1 release
That's exactly what I've been fighting this week. (kind of what I elaborated on the other day)
I'll update the make_release (to align to most of what Rafael's script has) and see how the strings look in the po files. The only other difference that we'll have afterwards is that make_release has the --no-wrap with msgmerge commands and his script does not. Since the translation was done on master, the GLTransInquiry file line references are off by a few lines.
Looking at some of the master PO files, the strings are split on multiple lines, which I thought that Raphael did not want to see?
I've not yet seen an email response, but I'd like to really get the release done this weekend, as this language matter is the only thing remaining. If necessary, I might have to revert his master commit so that I can re-run the language file on the branch to merge.
Anyway, will see how it goes tomorrow.
|
|
06-15-2019, 07:45 PM,
|
|
falkoner
Tim Schofield
   
|
Posts: 1,215
Threads: 17
Joined: Mar 2015
|
|
RE: Pending 4.15.1 release
(06-15-2019, 01:05 PM)TurboPT Wrote: If necessary, I might have to revert his master commit so that I can re-run the language file on the branch to merge.
This is a problem with not using Git/Github as it is meant to be used. We are trying to force a distributed version control system to behave as a centralised version control system, and in the process losing most of the advantages of a DVCS.
Nobody should be pushing directly to the webERP-team repository, they should be pushing to their own repository and then issuing a pull request to webERP-team. That way the commit can be reviewed before it ends up in the main line, where it has to be reversed out in a clumsy way if it is wrong.
|
|
06-16-2019, 05:09 PM,
|
|
falkoner
Tim Schofield
   
|
Posts: 1,215
Threads: 17
Joined: Mar 2015
|
|
RE: Pending 4.15.1 release
I agree Paul, let's just go for it.
Github provides a way of enforcing pull requests for main repositories but it is always best if these things can be done voluntarily. It is after all a volunteer project.
|
|
|