[Bash-completion-devel] [bash-completion-Bugs][313111] Please add scilab completion support

bash-completion-bugs at alioth.debian.org bash-completion-bugs at alioth.debian.org
Tue Mar 13 20:01:59 UTC 2012


Bugs item #313111, was changed at 2011-05-08 21:31 by Igor Murzov
You can respond by visiting: 
https://alioth.debian.org/tracker/?func=detail&atid=413095&aid=313111&group_id=100114

Status: Open
Priority: 3
Submitted By: Sylvestre Ledru (sylvestre)
Assigned to: Nobody (None)
Summary: Please add scilab completion support 
Distribution: --Distribution-Agnostic--
Originally reported in: Debian BTS
Milestone: None
Status: None
Original bug number: 575527


Initial Comment:
Hello,

Could you review and apply the attached patch which adds the Scilab completion ?

Thanks

----------------------------------------------------------------------

>Comment By: Igor Murzov (garik-guest)
Date: 2012-03-14 00:01

Message:
Hi, Sylvestre.

I reviewed your patch and it looks messy, but it's fixable. Anyway, why don't you ask scilab maintainers to include your completion into scilab itself?

----------------------------------------------------------------------

Comment By: Sylvestre Ledru (sylvestre)
Date: 2012-03-11 14:21

Message:
ping ?

----------------------------------------------------------------------

Comment By: Sylvestre Ledru (sylvestre)
Date: 2011-06-20 15:24

Message:
Thanks for your comments!

Here is an updated patch taking in account your comments.


----------------------------------------------------------------------

Comment By: Sylvestre Ledru (sylvestre)
Date: 2011-05-20 16:31

Message:
Hello,

thanks for all your comments!
I will have a look to fix all the issues.


----------------------------------------------------------------------

Comment By: Ville Skyttä  (scop-guest)
Date: 2011-05-17 21:26

Message:
Potential and actual problems:

1) If the text in the license header is correct and the file is indeed part of Scilab, I see little reason to include it also as part of bash-completion especially because it doesn't seem to be using any bash-completion features except _filedir.

2) Even though I guess the CeCILL v2 license and the GPL v2+ of bash-completion are compatible, the license header seems to imply that we should be including the CeCILL v2 license text also with bash-completion. I think it'd be better if the file was licensed under GPL v2+ like the rest of bash-completion if it is to be included.

3) The file should be changed to use _init_completion, see current bash-completion git.

4) Current bash-completion git no longer defines $filenames.  _filedir turns on -o filenames automatically when needed.

5) I don't think there's any reason to use grep with compgen -W, just use compgen -W "$options" -- "$cur"

6) There's no need to include upper and lowercase stuff in _filedir argument like the completion does, lowercase is enough.

7) Have you looked into using _parse_help or _parse_usage instead of hardcoding lists of available options?

8) "# List all options. Note that" -- note what?

----------------------------------------------------------------------

You can respond by visiting: 
https://alioth.debian.org/tracker/?func=detail&atid=413095&aid=313111&group_id=100114



More information about the Bash-completion-devel mailing list