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

[fix] filtron.sh: fix filtron proxy error #455

Merged
merged 1 commit into from Oct 29, 2021

Conversation

dalf
Copy link
Member

@dalf dalf commented Oct 29, 2021

What does this PR do?

build filtron with the golang version 1.17.2

Why is this change important?

#443 is fixed by upgrading the golang version.

How to test this PR locally?

In a new VM:

Author's checklist

Related issues

close #443

morty & filtron upgrade golang from v1.13.5 to v1.17.2 (#stable go[1])

Closes: searxng#443

[1] https://golang.org/dl/#stable
Copy link
Member

@return42 return42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, thanks a lot .. I also upgraded morty's golang from v1.13.5 to v1.17.2 and forced pushed on your branch dalf:fix-filtron-install.

@return42 return42 merged commit 5c4afdd into searxng:master Oct 29, 2021
@dalf dalf deleted the fix-filtron-install branch October 29, 2021 14:38
return42 added a commit to return42/searxng that referenced this pull request Oct 29, 2021
Related-to: 5c4afdd searxng#455

Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
return42 added a commit to return42/searxng that referenced this pull request Oct 29, 2021
Related-to: 5c4afdd searxng#455

Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
return42 added a commit that referenced this pull request Oct 29, 2021
Related-to: 5c4afdd #455

Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
return42 added a commit that referenced this pull request Oct 29, 2021
Related-to: 5c4afdd #455

Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
@tiekoetter
Copy link
Member

Note: When golang is installed via a package manager the local version, which filtron downloads, is ignored.

return42 added a commit to return42/searxng that referenced this pull request Oct 30, 2021
When golang is installed via a package manager the local version, which filtron
downloads, is ignored.

Reported-by: @tiekoetter searxng#455 (comment)
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
return42 added a commit to return42/searxng that referenced this pull request Oct 30, 2021
When golang is installed via a package manager the local version, which filtron
downloads, is ignored.

BTW: With the new go1.17.2 (cfea51f), 'go get' is depricated::

  go get: installing executables with 'go get' in module mode is deprecated.
    Use 'go install pkg@version' instead.
    For more information, see https://golang.org/doc/go-get-install-deprecation
    or run 'go help get' or 'go help install'.

Reported-by: @tiekoetter searxng#455 (comment)
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
@return42
Copy link
Member

Note: When golang is installed via a package manager the local version, which filtron downloads, is ignored.

@tiekoetter: thanks for the information, it is fixed in #461

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 this pull request may close these issues.

[question] filtron fails after reinstalation / in new installations
3 participants