Hello Laurent,
I agree with you, it may be preferable to keep completion for "ml"
simple. Maybe add completion for long option when "--" is entered.
Best regards,
Xavier
Le mer. 21 févr. 2024 à 13:04, Laurent BESSON
<lau...@ea...> a écrit :
>
> Hello,
>
> I have tried to add all possible completion from "module" command to "ml" command but, at the end, I found it confusing:
>
> Trying to hit "tab" after:
> >>>> ml -
>
> will list all already loaded modules but also all possible options of module command (like "--version"). So the display is a bit messy with a mix of loaded modules and options. I think we are losing the simplicity of the new "ml" command whose first intention is to list/load/unload.
>
> Same when trying to complete "ml " with either the available modules and all the commands of "module".
>
> However, as suggested, I'll fire a new request under GitHub with essential completion for ml
>
> BR,
> Laurent.
>
>
> -----Message d'origine-----
> De : Xavier Delaruelle <xav...@gm...>
> Envoyé : mardi 20 février 2024 18:53
> À : Environment Modules usage and discussion. <mod...@li...>
> Objet : Re: [Modules] Add completetion for "ml" command
>
> Hello Laurent,
>
> It would be very interesting to have ml completion added for tcsh shell. Would you like to contribute and create a pull request for that?
>
> init/tcsh_completion.in file should be updated with a "complete ml"
> definition. From my understanding, this definition should equal the content of "complete module" + definition you propose.
>
> Best regards,
> Xavier
>
> Le mar. 20 févr. 2024 à 16:57, Laurent BESSON <lau...@ea...> a écrit :
> >
> > Hello,
> >
> >
> >
> > Would it be possible to add shell completion for the almost-new « ml » command.
> >
> > On my side, using tcsh as my loggin shell, I have defined the following :
> >
> >
> >
> > complete ml 'c/-/`_module_loaded`/' \
> >
> > 'p/1/`_module_not_yet_loaded; echo "-"`/'
> >
> >
> >
> > Which is working quite well.
> >
> >
> >
> > Best regards,
> >
> > Laurent.
|