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

Hugo Beauzée-Luyssen hugo at beauzee.fr
Tue Apr 10 18:30:16 CEST 2018


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


More information about the vlc-devel mailing list