Summary: | app-eselect/eselect-php - arguments to `eselect php' do not stroke with other eselect modules' arguments | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Jelte Fennema <gentoobugzilla> |
Component: | [OLD] Development | Assignee: | PHP Bugs <php-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | elis.axelsson |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://github.com/fish-shell/fish-shell/pull/1131#issuecomment-28877159 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Jelte Fennema
2013-11-21 12:50:17 UTC
I have the feeling this is not going to be fixed/changed any time in the foreseeable future. (In reply to Jelte Fennema from comment #1) > I have the feeling this is not going to be fixed/changed any time in the > foreseeable future. We can't really make it work exactly like all of the other eselect modules, since we're able to set the CLI PHP different from e.g. the apache2 PHP. So we need that "cli" or "apache2" somewhere in the eselect command. I always forget how to invoke it, too. I did give you list-modules though: $ sudo eselect php list-modules cli apache2 fpm cgi phpdbg That should be in the latest eselect-php-0.9.1 that I just pushed. It's masked for testing, but please give it a try. > (In reply to Michael Orlitzky from comment #2)
> I did give you list-modules though:
>
> $ sudo eselect php list-modules
> cli apache2 fpm cgi phpdbg
>
> That should be in the latest eselect-php-0.9.1 that I just pushed. It's
> masked for testing, but please give it a try.
Testing this at the moment, list-modules seems to work, but not entierly as one could expect since it also lists modules not installed/built.
For example, I only have php built with CLI, and still get all the modules.
But this is good enough for us to write dynamic completions, which is beeing done at the moment.
But it would be even better if it didn't print modules that can't be used for one reason or another. If there's no selection available, is that module available then? In current build it is.
But the new completions will depend on list-modules because it's good enough.
(In reply to Elis Axelsson from comment #3) > > (In reply to Michael Orlitzky from comment #2) > > I did give you list-modules though: > > > > $ sudo eselect php list-modules > > cli apache2 fpm cgi phpdbg > > > > That should be in the latest eselect-php-0.9.1 that I just pushed. It's > > masked for testing, but please give it a try. > > Testing this at the moment, list-modules seems to work, but not entierly as > one could expect since it also lists modules not installed/built. > > For example, I only have php built with CLI, and still get all the modules. > Yeah, they're all still valid arguments to `eselect php <action>` though, so they should show up in the tab-completion as long as we accept them. Useless, but valid: $ sudo eselect php list phpdbg (none found) And there are invalid arguments, so there is a distinction: $ sudo eselect php list derp !!! Error: Please choose one of the following modules: cli apache2 fpm cgi phpdbg exiting Maybe it would be nicer to throw an error if there are no targets -- that would let us trim the list of valid SAPIs -- but it's probably too late in the game to mess with the API without a good reason. I'm going to resolve this because I think I've done as much as I can without breaking our API. If you need anything else, please don't hesitate to open a new bug. |