zdharma-continuum/community

zdharma-continuum's main town hall!

Stefanqn

@Stefanqn

@vladdoster that's what I do

zinit wait lucid is-snippet \
  id-as'pip_completion' atclone'pip completion --zsh >! _pip' atpull"%atclone" atload'source _pip' for /dev/null

vladislav doster

@vladdoster

i need to do work (last day of the sprint :P) on a story, but i'll look at the other 2 later

the source lets you skip needing _pip file at all

Stefanqn

@Stefanqn

I personally think, that the handling of /dev/null for snippets changed, so I get this error message now

vladislav doster

@vladdoster

but it isnt a snippet

Stefanqn

@Stefanqn

yeah, but I create the completions just once at pull

it's an empty snippet :)

vladislav doster

@vladdoster

I'm just trying to help, but might start here

zdharma-continuum/zinit@b6f286b

Anupam Srivastava

@anupamsr

I am trying to use zinit on msys it seems to fail because of some quoting issue:
```$ zini for from"gh-r" lbin"jq-* -> jq" stedolan/jq
Downloading stedolan/jq…
(Requesting 'jq-win32.exe'')

ziextract: Unpacking the files from: 'jq-win32.exe'…
jq: error: syntax error, unexpected ':', expecting $end (Windows cmd shell quoting issues?) at <top-level>, line 1:
S:/
jq: 1 compile error
ziextract: WARNING: extraction of the archive 'jq-win32.exe' had problems.
```

Anupam Srivastava

@anupamsr

any idea? Is this supported usecase?

Anupam Srivastava

@anupamsr

nvm. the problem is that it is trying to extract from what is not, in fact, an archive

vladislav doster

@vladdoster

See https://github.com/vladdoster/dotfiles/blob/455560bbf3eb1b065404b28550e9445f7abfc56c/zsh/.config/zsh/zinit.zsh#L80

Anupam Srivastava

@anupamsr

Yes, but it seems it really fails for windows too

zini for from"gh-r" as"program" mv"docker* -> docker-compose" bpick"*win*" docker/compose

Downloading docker/compose…
(Requesting `docker-compose-windows-x86_64.exe'…)

ziextract: Unpacking the files from: `docker-compose-windows-x86_64.exe'…

Usage:  docker compose [OPTIONS] COMMAND

Docker Compose

Options:
      --ansi string                Control when to print ANSI control
                                   characters ("never"|"always"|"auto")
                                   (default "auto")
      --compatibility              Run compose in backward compatibility mode
      --env-file string            Specify an alternate environment file.
  -f, --file stringArray           Compose configuration files
      --profile stringArray        Specify a profile to enable
      --project-directory string   Specify an alternate working directory
                                   (default: the path of the, first
                                   specified, Compose file)
  -p, --project-name string        Project name

Commands:
  build       Build or rebuild services
  convert     Converts the compose file to platform's canonical format
  cp          Copy files/folders between a service container and the local filesystem
  create      Creates containers for a service.
  down        Stop and remove containers, networks
  events      Receive real time events from containers.
  exec        Execute a command in a running container.
  images      List images used by the created containers
  kill        Force stop service containers.
  logs        View output from containers
  ls          List running compose projects
  pause       Pause services
  port        Print the public port for a port binding.
  ps          List containers
  pull        Pull service images
  push        Push service images
  restart     Restart containers
  rm          Removes stopped service containers
  run         Run a one-off command on a service.
  start       Start services
  stop        Stop services
  top         Display the running processes
  unpause     Unpause services
  up          Create and start containers
  version     Show the Docker Compose version information

Run 'docker compose COMMAND --help' for more information on a command.
unknown docker command: "compose S:/"
ziextract: WARNING: extraction of the archive `docker-compose-windows-x86_64.exe' had problems.

Just changing zini for from"gh-r" as"program" mv"docker -> docker-compose" bpick"*win*" docker/compose to bpick"*lin*" works

vladislav doster

@vladdoster

It’s because docker compose isn’t docker-compose?

You shouldn’t need bpick

See https://github.com/zdharma-continuum/zinit/blob/fb3c082551ee9f48676c3183d5a4e9e842d3d874/tests/gh-r.zunit#L153

Anupam Srivastava

@anupamsr

That is from https://github.com/zdharma-continuum/zinit example. Removing bpick doesn't help

I think the problem is indeed in ziextract: Unpacking the files from: 'docker-compose-windows-x86_64.exe'…

vladislav doster

@vladdoster

zi as'null' from"gh-r" lbin'!* -> docker-compose' lucid nocompile nocompletions for @docker/compose

Anupam Srivastava

@anupamsr

Same

vladislav doster

@vladdoster

What’s full output

Anupam Srivastava

@anupamsr

zini as'null' from"gh-r" lbin'!* -> docker-compose' lucid nocompile nocompletions for @docker/compose

Downloading docker/compose…
(Requesting `docker-compose-windows-x86_64.exe'…)

ziextract: Unpacking the files from: `docker-compose-windows-x86_64.exe'…

Usage:  docker compose [OPTIONS] COMMAND

Docker Compose

Options:
      --ansi string                Control when to print ANSI control
                                   characters ("never"|"always"|"auto")
                                   (default "auto")
      --compatibility              Run compose in backward compatibility mode
      --env-file string            Specify an alternate environment file.
  -f, --file stringArray           Compose configuration files
      --profile stringArray        Specify a profile to enable
      --project-directory string   Specify an alternate working directory
                                   (default: the path of the, first
                                   specified, Compose file)
  -p, --project-name string        Project name

Commands:
  build       Build or rebuild services
  convert     Converts the compose file to platform's canonical format
  cp          Copy files/folders between a service container and the local filesystem
  create      Creates containers for a service.
  down        Stop and remove containers, networks
  events      Receive real time events from containers.
  exec        Execute a command in a running container.
  images      List images used by the created containers
  kill        Force stop service containers.
  logs        View output from containers
  ls          List running compose projects
  pause       Pause services
  port        Print the public port for a port binding.
  ps          List containers
  pull        Pull service images
  push        Push service images
  restart     Restart containers
  rm          Removes stopped service containers
  run         Run a one-off command on a service.
  start       Start services
  stop        Stop services
  top         Display the running processes
  unpause     Unpause services
  up          Create and start containers
  version     Show the Docker Compose version information

Run 'docker compose COMMAND --help' for more information on a command.
unknown docker command: "compose S:/"
ziextract: WARNING: extraction of the archive `docker-compose-windows-x86_64.exe' had problems.

vladislav doster

@vladdoster

Try for lbin”!* -> docker-compose.exe”

Does windows have symlinks? I haven’t used it in forever

Or change lbin to mv

Anupam Srivastava

@anupamsr

Hmm

vladislav doster

@vladdoster

It obviously works, but it’s calling it on when running one of the ices

Anupam Srivastava

@anupamsr

It does, in a way

I tried mv earlier btw, and now again. No help

+→zinit-extract:1> chmod a+x -- ./docker-compose-windows-x86_64.exe
+→zinit-extract:2> cygpath -w /home/srivastavaan/.local/share/zinit/plugins/docker---compose
+→zinit-extract:2> ./docker-compose-windows-x86_64.exe /S '/D=E:\scoop\apps\msys2\2022-06-03\home\srivastavaan\.local\share\zinit\plugins\docker---compose'

This is the issue

vladislav doster

@vladdoster

Where is /S coming from

Anupam Srivastava

@anupamsr

1814         ((#i)*.exe|(#i)*.pe32)
1815             →zinit-extract() {
1816                 command chmod a+x -- ./$file
1817                 ./$file /S /D="`cygpath -w $PWD`"
1818             }
1819             ;;

this is from zinit-install.zsh

vladislav doster

@vladdoster

Can you open an issue so I can track this?

Anupam Srivastava

@anupamsr

No idea what it is supposed to do: commit id c1163ff3abf38fd99e925581041c5499e539e6aa

vladislav doster

@vladdoster

Also, what happens if you only use “

Or vice versa

‘

Might try that

Anupam Srivastava

@anupamsr

Oh. So double quote is probably not a good idea because it expands wildcards, but it works in .zshrc and satisfies my ocd of shellchek :)

come to think of it, i suppose it is best to use single quote everywhere

*when using zinit

Anupam Srivastava

@anupamsr

zdharma-continuum/zinit#306

vladislav doster

@vladdoster

Thanks @Anupam

Sebastian Gniazdowski

@psprint

@vladdoster @anupamsr The /S /D comes from Firefox windows installer.. Sorry for not commenting this, that what the *.exesupport in ziextractis is actually Firefox installer support, experimental proof of concept to introduce a more versatile support later.. This is what makes pack for firefox on windows work

vladislav doster

@vladdoster

@psprint thanks for the insight on the issue

I haven’t used windows in over a decade so I’m getting up to speed with windows related Zinit related code

andriy melnyk

@turboBasic

+→zinit-extract:1> chmod a+x -- ./docker-compose-windows-x86_64.exe
+→zinit-extract:2> cygpath -w /home/srivastavaan/.local/share/zinit/plugins/docker---compose
+→zinit-extract:2> ./docker-compose-windows-x86_64.exe /S '/D=E:\scoop\apps\msys2\2022-06-03\home\srivastavaan\.local\share\zinit\plugins\docker---compose'

btw, how did you enable this log output?

vladislav doster

@vladdoster

@turboBasic I don't follow your question?

Where communities thrive

People

Repo info

Activity