usage of asset_filter breaks target

[leapfog]

With the following simple config file

[global]
  target = "~/bin"
  upgrade_only = true

the command eget neovim/neovim downloads the appimage and places it in ~/bin.

But when I add

["neovim/neovim"]
   asset_filters = "nvim.appimage"

the download ends up in ./ (i.e. the current directory).

[patinthehat]

@leapfog It looks like this might be a bug related to the expansion of the tilde (~). Can you verify that it works correctly if you use an absolute pathname in the [global] section for target?

i.e.,

[global]
  target = "/home/someuser/bin"
  upgrade_only = true

[leapfog]

No, that doesn't help.

But testing that, it seems, that adding asset_filters = "nvim.appimage" simply (and silently!) breaks reading the global section config file.

update:

using asset_filters = [ "nvim.appimage" ] makes the whole thing work.

but also #65 happens here: nvim is downloaded/overwritten every time.

[patinthehat]

@leapfog Does using the array syntax fix this? Just trying to narrow down the possibilities here.

["neovim/neovim"]
   asset_filters = [ "nvim.appimage" ]

[hhromic]

I think this is an effect of what I just reported in #84. When a configuration file is malformed, eget simply ignores it and uses defaults if no other valid config files are found. The default for target is to save the binaries in the current directory.

In that isssue I'm also proposing for eget to report configuration file errors instead of ignoring them. This would make this kind of issues very visible and less confusing.

[zyedidia]

I believe this is fixed by #85.