[vlc-devel] [vlc-commits] lua: libs: Add a vlc.io and move mkdir there

[Rémi Denis-Courmont]

Le 10 avril 2018 19:30:16 GMT+03:00, "Hugo Beauzée-Luyssen" <hugo at beauzee.fr> a écrit :
>On Mon, Apr 9, 2018, at 10:31 PM, Rémi Denis-Courmont wrote:
>> Directory exists is not the same as directory was created.
>Permissions 
>> may not match the request, and directory might not be empty.
>> 
>> This also does not match any common semantics. Regular mkdir fails if
>
>> directory exists, while mkdir -p succeeds, but is recursive.
>> 
>> And I know that this is a preexisting design matter but if the API is
>
>> changed (renamed), it might as well be fixed.
>> -- 
>> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez
>excuser 
>> ma brièveté.
>> _______________________________________________
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
>> https://mailman.videolan.org/listinfo/vlc-devel
>
>Would the proposed patch work for you? I'm a bit reluctant to expose
>errno to lua scripts...
>We might also want to add a boolean parameter to vlc.io.mkdir, that
>would be helpful, and we wouldn't have to return 2 booleans.
>
>Regards,
>
>-- 
>  Hugo Beauzée-Luyssen
>  hugo at beauzee.fr
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel

I don't have any bright idea. It feels like returning different integer/error values is the least bad.
-- 
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.