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

Building Gitea will build go-sqlite3 with or without the sqlite build tag #2172

Closed
3 of 9 tasks
dubeg opened this issue Jul 17, 2017 · 2 comments · Fixed by #2177
Closed
3 of 9 tasks

Building Gitea will build go-sqlite3 with or without the sqlite build tag #2172

dubeg opened this issue Jul 17, 2017 · 2 comments · Fixed by #2177
Labels
topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile type/bug
Milestone

Comments

@dubeg
Copy link
Contributor

dubeg commented Jul 17, 2017

  • Gitea version (or commit ref): master
  • Git version: 2.8.1
  • Operating system: Windows 10
  • Database:
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
    • Not relevant
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:
    • Not relevant

Description

Even without specifying "sqlite" as a build tag, the command go build main.go will also build the vendor go-sqlite3, which requires gcc.

Screenshots

sqlitebuild

Note

Currently, two of Gitea's files import this vendor: models/models_sqlite.go and models/unit_tests.go. The former has the sqlite build constraint, while the latter does not. This is probably the root of this issue.

@go2sh
Copy link

go2sh commented Jul 17, 2017

The sqlite tag only touches the final product not the tests. One could argue to add a different command for building the tests. But this an enhancement not a bug.

@lafriks
Copy link
Member

lafriks commented Jul 17, 2017

@go2sh if it is not possible to build gitea without gcc than it is a bug

@lafriks lafriks added type/bug topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile labels Jul 17, 2017
@lunny lunny added this to the 1.2.0 milestone Jul 26, 2017
@cez81 cez81 mentioned this issue Dec 9, 2017
7 tasks
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile type/bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants