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

Please add support for LibreWolf #79

Open
NuLL3rr0r opened this issue Jul 2, 2022 · 2 comments · May be fixed by #110
Open

Please add support for LibreWolf #79

NuLL3rr0r opened this issue Jul 2, 2022 · 2 comments · May be fixed by #110

Comments

@NuLL3rr0r
Copy link

NuLL3rr0r commented Jul 2, 2022

Summary

For privacy concerns, I've switched from Brave to LibreWolf, which is a solid Firefox/Brave alternative without the annoyances of Firefox. I've noticed gopass-jsonapi is not able to set up the Gopass bridge.

Steps To Reproduce

!. Install LibreWolf
2. Install Gopass extension
3. Try to configure using gopass-jsonapi

Expected behavior

LibreWolf is not different from Firefox that much, except for being hardened for privacy. This should be easy to set up and work.

Environment

  • OS: Gentoo
  • OS version: Linux mamadou-legion 5.15.50 # 34 SMP Tue Jun 28 16:56:47 CEST 2022 x86_64 11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz GenuineIntel GNU/Linux
  • gopass-jsonapi Version: gopass-jsonapi version 1.11.1
  • Installation method: My own custom ebuilds

Additional context

BTW, I've noticed the version is being reported as 1.11.1 by gopass-jsonapi. Looking at the repository/VERSION file it seems it contains the wrong version string. I'm running 1.14.3 at the moment.

@dominikschulz
Copy link
Member

Thanks for the suggestion. Sounds like a valid feature request.

But the version issue should be tracked separately.

@NuLL3rr0r
Copy link
Author

For the version, it should be possible to extract the git tag inside the Makefile from Git executable. If that sounds like a good idea, I could make a pull request for it.

This is what, I usually do:

GIT_BRANCH	:=	$(shell git rev-parse --abbrev-ref HEAD)
GIT_TAG		:=	$(shell git describe --tags --abbrev=0 2>/dev/null)
GIT_COMMIT	:=	$(shell git rev-parse --short HEAD)
GIT_TREE_STATE	:=	$(shell test -n "`git status --porcelain`" && echo "dirty" || echo "clean")

ifeq ($(GIT_BRANCH),master)
ifeq ($(strip $(GIT_TAG)),)
ifeq ($(GIT_TREE_STATE),dirty)
VERSION := v0.0.0-${GIT_COMMIT}*
else # ($(GIT_TREE_STATE),dirty)
VERSION := v0.0.0-${GIT_COMMIT}
endif # ($(GIT_TREE_STATE),dirty)
else # ($(strip GIT_TAG),)
ifeq ($(GIT_TREE_STATE),dirty)
VERSION := ${GIT_TAG}-${GIT_COMMIT}*
else # ($(GIT_TREE_STATE),dirty)
VERSION := ${GIT_TAG}-${GIT_COMMIT}
endif # ($(GIT_TREE_STATE),dirty)
endif # ($(strip GIT_TAG),)
else # ($(GIT_BRANCH),master)
ifeq ($(GIT_TREE_STATE),dirty)
VERSION := ${GIT_BRANCH}-${GIT_COMMIT}*
else # ($(GIT_TREE_STATE),dirty)
VERSION := ${GIT_BRANCH}-${GIT_COMMIT}
endif # ($(GIT_TREE_STATE),dirty)
endif # ($(GIT_BRANCH),master)

VERSION_MAJOR = $(shell echo ${GIT_TAG} | cut -d '.' -f 1 | tr -d -c 0-9)
ifeq ($(VERSION_MAJOR),)
VERSION_MAJOR := 0
endif

VERSION_MINOR = $(shell echo ${GIT_TAG} | cut -d '.' -f 2 | tr -d -c 0-9)
ifeq ($(VERSION_MINOR),)
VERSION_MINOR := 0
endif

VERSION_PATCH = $(shell echo ${GIT_TAG} | cut -d '.' -f 3 | tr -d -c 0-9)
ifeq ($(VERSION_PATCH),)
VERSION_PATCH := 0
endif

@samuelnilsson samuelnilsson linked a pull request Sep 7, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants