Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

exec: Failed to execute process '(null)': An argument exceeds the OS argument length limit. #6

Closed
jose1711 opened this issue Jul 5, 2022 · 1 comment · Fixed by #7
Assignees
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@jose1711
Copy link

jose1711 commented Jul 5, 2022

Describe the bug
Receiving an error when using to complete pacman -S arguments.

To Reproduce

  • install fifc
  • type pacman -S , hit shortcut to invoke fifc

Expected behavior
Package name(s) should appear

Actual behavior

pacman -S exec: Failed to execute process '(null)': An argument exceeds the OS argument length limit.                                     
exec: Failed to execute process '(null)': An argument exceeds the OS argument length limit.
exec: Failed to execute process '(null)': An argument exceeds the OS argument length limit.
exec: Failed to execute process '(null)': An argument exceeds the OS argument length limit.
exec: Failed to execute process '(null)': An argument exceeds the OS argument length limit.
exec: Failed to execute process '(null)': An argument exceeds the OS argument length limit.
exec: Failed to execute process '(null)': An argument exceeds the OS argument length limit.

System (please complete the following information):

  • fish version 3.4.1
  • Arch Linux x86_64 rolling

Additional context
If I specify a more narrow search, like pacman -S firef no error appears - instead a list of packages starting with firef are shown.

@jose1711 jose1711 added the bug Something isn't working label Jul 5, 2022
@gazorby
Copy link
Owner

gazorby commented Jul 6, 2022

Hi @jose1711!

This seems related to #3. It's due to the fact that when completion is triggered, we run complete -C first, to determine the suggestion group and store the results in a list, which can leads to big OS argument size.

#4 was a quick fix, only solving the special case of empty commandline, which is likely to produce large number of results, but we need a more robust solution.

@gazorby gazorby added the good first issue Good for newcomers label Jul 6, 2022
@gazorby gazorby mentioned this issue Jul 7, 2022
@gazorby gazorby closed this as completed in #7 Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants