Skip to content

Fix compiling without sqlite and gcc - #2177

Merged
lunny merged 1 commit into
go-gitea:masterfrom
lafriks-fork:fix/build_without_sqlite
Jul 26, 2017
Merged

Fix compiling without sqlite and gcc#2177
lunny merged 1 commit into
go-gitea:masterfrom
lafriks-fork:fix/build_without_sqlite

Conversation

@lafriks

@lafriks lafriks commented Jul 17, 2017

Copy link
Copy Markdown
Member

Fixes #2172

@lafriks lafriks added the topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile label Jul 17, 2017
@lunny lunny added this to the 1.2.0 milestone Jul 18, 2017
@lunny lunny added the type/bug label Jul 18, 2017
@lunny

lunny commented Jul 18, 2017

Copy link
Copy Markdown
Member

No. the build tag is not in the correct place

@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 18, 2017
@lafriks
lafriks force-pushed the fix/build_without_sqlite branch from d0a5416 to ccde6b8 Compare July 18, 2017 05:21
@lafriks

lafriks commented Jul 18, 2017

Copy link
Copy Markdown
Member Author

@lunny sorry that build tag was leftover from first try to fix compilation with no gcc, it was not needed, fixed

@lafriks
lafriks force-pushed the fix/build_without_sqlite branch from ccde6b8 to 82f5283 Compare July 18, 2017 13:46
@lunny

lunny commented Jul 19, 2017

Copy link
Copy Markdown
Member

maybe rename models/unit_tests.go to models/unit_test.go is better?

@ethantkoenig

Copy link
Copy Markdown
Member

@lunny No, it is important that models/unit_tests.go is not treated as a test file; otherwise the integration tests will not be able to import it.

@ethantkoenig

Copy link
Copy Markdown
Member

LGTM

@tboerger tboerger added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jul 19, 2017
@Bwko

Bwko commented Jul 22, 2017

Copy link
Copy Markdown
Member

LGTM

@lafriks lafriks added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jul 24, 2017
@lunny

lunny commented Jul 26, 2017

Copy link
Copy Markdown
Member

make CI work

@lunny
lunny merged commit 1d032f5 into go-gitea:master Jul 26, 2017
@cez81 cez81 mentioned this pull request Dec 9, 2017
7 tasks
@lafriks
lafriks deleted the fix/build_without_sqlite branch December 10, 2017 10:08
@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

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

5 participants