Skip to content

Make bindata.go optional, remove from repository #172

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from

Conversation

strk
Copy link
Member

@strk strk commented Nov 14, 2016

By default will read assets from filesystems, but can embed them
by passing the "bindata" build tag (go build -tags bindata)

Fixes #30

Print a WARNING when bindata is not embedded, and
honours GITEA_WORK_DIR, when given, for finding filesystem assets.

Includes automatic test for the dynamic "bin"data library.

NOTE: an equivalent functionality is planned to be included in a
bigger change by @tboerger currently in progress and tracked as PR #74;
this PR is smaller in scope and ready.

Please keep open until either this or #74 is merged.

@bkcsoft
Copy link
Member

bkcsoft commented Nov 14, 2016

Keep this open til #74 is merged, so people can use this before the other fix is done.

@bkcsoft bkcsoft added type/enhancement An improvement of existing functionality issue/duplicate The issue has already been reported. labels Nov 14, 2016
@codecov-io
Copy link

codecov-io commented Nov 14, 2016

Current coverage is 3.25% (diff: 31.25%)

Merging #172 into master will increase coverage by 0.22%

@@            master      #172   diff @@
========================================
  Files           33        34     +1   
  Lines         8106      8170    +64   
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
+ Hits           246       266    +20   
- Misses        7840      7881    +41   
- Partials        20        23     +3   

Powered by Codecov. Last update 2ccdcda...38b713c

@strk strk force-pushed the optional-bindata branch 4 times, most recently from 8213590 to 9184ca6 Compare November 21, 2016 10:07
@strk strk force-pushed the optional-bindata branch 3 times, most recently from 8333ffc to 3d14719 Compare November 24, 2016 08:02
@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 24, 2016
@strk strk force-pushed the optional-bindata branch 5 times, most recently from 25ceda1 to 5ff07a9 Compare November 27, 2016 11:00
@tboerger tboerger added this to the 1.0.0 milestone Nov 29, 2016
@strk strk force-pushed the optional-bindata branch 3 times, most recently from b324f98 to 8f6c1e8 Compare December 2, 2016 11:06
@bkcsoft bkcsoft removed this from the 1.0.0 milestone Dec 3, 2016
By default will read assets from filesystems, but can embed them
by passing the "bindata" build tag (go build -tags bindata)

Fixes gogs#77

Print a WARNING when bindata is not embedded, and
honours GITEA_WORK_DIR, when given, for finding filesystem assets.

Include automatic test for the dynamic "bin"data library
@strk strk force-pushed the optional-bindata branch from 8f6c1e8 to c155be7 Compare December 3, 2016 11:38
@tboerger
Copy link
Member

tboerger commented Dec 5, 2016

Obsoleted via #354

@tboerger tboerger closed this Dec 5, 2016
@tboerger tboerger removed type/enhancement An improvement of existing functionality lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Dec 5, 2016
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
issue/duplicate The issue has already been reported.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Embed bindata optionally
4 participants