gopass find includes symlinks

[532910]

As the pass itself and buggy tree,

% gopass find linux
...password-store
├── IT
│   └── linux.org.ru.gpg
└── shops
    ├── amazon.ca.gpg -> amazon.gpg
    ├── amazon.co.uk.gpg -> amazon.gpg
    ├── amazon.com.gpg -> amazon.gpg
    ├── amazon.de.gpg -> amazon.gpg
    ├── ebay.ca.gpg -> ebay.gpg
    ├── ebay.co.uk.gpg -> ebay.gpg

[dominikschulz]

I'm not sure if I understand correctly, but gopass is supposed to include symlinks.
What would you expect?

[532910]

gopass find linux must not show amazon

[dominikschulz]

I'm afraid, but it still don't get it. Maybe you could provide a listing of the directory on question along with the actual and expected outputs.

[532910]

Real output:

% gopass find linux
/home/sergio/.password-store
├── IT
│   └── linux.org.ru.gpg
└── shops
    ├── amazon.ca.gpg -> amazon.gpg
    ├── amazon.co.uk.gpg -> amazon.gpg
    ├── amazon.com.gpg -> amazon.gpg
    ├── amazon.de.gpg -> amazon.gpg
    ├── ebay.ca.gpg -> ebay.gpg
    └── ebay.co.uk.gpg -> ebay.gpg

Expected output:

% gopass find linux
/home/sergio/.password-store
└── IT
    └── linux.org.ru.gpg

[luxalpa]

Why does it show amazon? I can't reproduce this issue :<

[532910]

As amazon is a symlink.

[532910]

It's tree bug
https://lists.zx2c4.com/pipermail/password-store/2018-April/003256.html
https://lists.zx2c4.com/pipermail/password-store/2018-April/003257.html

[dominikschulz]

gopass doesn't use tree.

[532910]

So it's another bug. Anyway, could you reproduce it?

[dominikschulz]

I'm afraid I wasn't able to reproduce it.

[532910]

I'm sorry. It's bug of 1.2.0-1 from debian sid
All works fine with 1.7.2

[falschparker82]

@c-e-p-x-u-o - note that the Debian package is a completely independent version of gopass from Alexandre Viau: https://salsa.debian.org/debian/gopass

It does not have anything to do with https://github.com/gopasspw/gopass

[dominikschulz]

In fact we aware of this naming conflict and have been in contact with the Debian maintainer of the other gopass package in Debian. Unfortunately we could not reach any compromise that would have made this situation obvious to users.

I'd suggest you raise an issue the Debian BTS to ask for clarification.

[532910]

Pleas add a big note about name conflict to not confuse users.

[dewey]

It's all in the installation description: https://github.com/gopasspw/gopass/blob/master/docs/setup.md#ubuntu--debian-1

I'm not sure if users who blindly install something would read the note if there was one.

[dominikschulz]

@dewey I fully agree, but we should probably at least add a small note about this naming issue on that page.

[532910]

You mean compromise with Alexandre Viau?

[532910]

The note should be on the main page.

[dominikschulz]

Yes, we talked to him but couldn't reach any working compromise with him.

No, we probably won't put this on the front page. The setup instructions shuld be good.

[luxalpa]

The information about the other package with the same name could be added to the Issue Template?

[dominikschulz]

@destructive-dragon Good idea, but we don't have an issue template, yet.
I'll add one.