[Caml-list] Support de l'UTF16

[Caml-list] Support de l'UTF16

[Clément Franchini]

Bonjour.
J'écris suite à mon message sur le ticket
https://caml.inria.fr/mantis/view.php?id=3771

J'ai passé pas mal de temps sur le support des chemins de fichier en
utf16 sous NT. J'ai déjà soumis trois patchs (en fonction du commit sur
lequel on veut appliquer le patch).

Nous, Altair, sommes intéressés par l'intégration de ce commit/patch
dans OCaml. L'idéal serait une intégration dans la branche 4.04, mais je
ne pense pas que cela soit possible.

Ce n'est pas un petit commit, il nécessite d'être revu. J'écris ce mail
pour faire avancer les choses. Nous y avons grand intérêt.

Cordialement/Regards.
--
Clément Franchini

RE: [Caml-list] Support de l'UTF16

[David Allsopp]

Clément Franchini wrote:
> Bonjour.
> J'écris suite à mon message sur le ticket
> https://caml.inria.fr/mantis/view.php?id=3771
> 
> J'ai passé pas mal de temps sur le support des chemins de fichier en
> utf16 sous NT. J'ai déjà soumis trois patchs (en fonction du commit sur
> lequel on veut appliquer le patch).
> 
> Nous, Altair, sommes intéressés par l'intégration de ce commit/patch dans
> OCaml. L'idéal serait une intégration dans la branche 4.04, mais je ne
> pense pas que cela soit possible.
> 
> Ce n'est pas un petit commit, il nécessite d'être revu. J'écris ce mail
> pour faire avancer les choses. Nous y avons grand intérêt.
> 
> Cordialement/Regards.

I saw the patch arrive last week, but I'm afraid I hadn't had a chance to look into it. Is this patch a rebase of the existing ones on that Mantis PR, or a completely new one (see also https://github.com/ocaml/ocaml/pull/153)?

It's not compulsory, but it might be easier to review this as a GitHub pull request. It's fine to be working on trunk only for this change - I'm afraid it's very unlikely that it would be officially back-ported to 4.04 or 4.05.


David

Re: [Caml-list] Support de l'UTF16

[Clément Franchini]

Hi,
sorry for my first mail in French.

> or a completely new one (see also https://github.com/ocaml/ocaml/pull/153
It's that pull request, but the 8 commits are merged/squashed into one.
And I had to change a big part of it, because the OCaml compiler has
evolved since 4.02.3. It was not a trivial rebase, the initial patch
need some work to be rebased.

> GitHub pull request
Today I don't know if I can have a github account. with my professional
email. I asked if I can, I'm waiting the answer.

Regards.
--
Clément Franchini

On 2017-05-24 18:02, David Allsopp wrote:
> Clément Franchini wrote:
>> Bonjour.
>> J'écris suite à mon message sur le ticket
>> https://caml.inria.fr/mantis/view.php?id=3771
>>
>> J'ai passé pas mal de temps sur le support des chemins de fichier en
>> utf16 sous NT. J'ai déjà soumis trois patchs (en fonction du commit sur
>> lequel on veut appliquer le patch).
>>
>> Nous, Altair, sommes intéressés par l'intégration de ce commit/patch dans
>> OCaml. L'idéal serait une intégration dans la branche 4.04, mais je ne
>> pense pas que cela soit possible.
>>
>> Ce n'est pas un petit commit, il nécessite d'être revu. J'écris ce mail
>> pour faire avancer les choses. Nous y avons grand intérêt.
>>
>> Cordialement/Regards.
> 
> I saw the patch arrive last week, but I'm afraid I hadn't had a chance to look into it. Is this patch a rebase of the existing ones on that Mantis PR, or a completely new one (see also https://github.com/ocaml/ocaml/pull/153)?
> 
> It's not compulsory, but it might be easier to review this as a GitHub pull request. It's fine to be working on trunk only for this change - I'm afraid it's very unlikely that it would be officially back-ported to 4.04 or 4.05.
> 
> 
> David
>