Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upEvaluate alternative static file embedder #2411
Comments
This comment has been minimized.
This comment has been minimized.
tamalsaha
commented
Feb 10, 2017
|
Hi,
|
This comment has been minimized.
This comment has been minimized.
|
Interesting. But I'm assuming that this timestamp is used in HTTP header fields, which in return are used to do cache invalidation in the browser so users actually see changes happening. |
This comment has been minimized.
This comment has been minimized.
tamalsaha
commented
Feb 10, 2017
•
|
I see. In our usecase w don't serve them from via HTTP. So, it was not an issue. go-bindata sets this timestamp from the actual file timestamp. So, sounds like you want to have the timestamp. |
This comment has been minimized.
This comment has been minimized.
|
I tried statik and can confirm it also uses modification-date from filesystem so it has the same "issue". go-bintool fixed timestamp parameter generated from the last commit timestamp on web/ui/static might be viable idea. But i'm not sure if that's possible in build environment. |
This comment has been minimized.
This comment has been minimized.
|
A related improvement was proposed to bindata but seems dismissed without any progress. https://github.com/jteeuwen/go-bindata/issues/120 |
brian-brazil
added
priority/Pmaybe
component/ui
kind/enhancement
labels
Jul 14, 2017
This comment has been minimized.
This comment has been minimized.
|
Also, BTW: Currently, FWIW: Yesterday, I was searching for a designated successor to
|
brian-brazil
added
help wanted
priority/P3
and removed
priority/Pmaybe
labels
Jul 12, 2018
This comment has been minimized.
This comment has been minimized.
|
Thanks for the information, that bumps the priority. If someone could help with this that'd be great, we've struggled with getting this right a fair bit over the years. |
This comment has been minimized.
This comment has been minimized.
|
Head-up: I'm looking into it right now and https://github.com/shurcooL/vfsgen looks promising. |
simonpasquier
self-assigned this
Jul 27, 2018
This was referenced Jul 27, 2018
grobie
closed this
in
#4430
Aug 24, 2018
This comment has been minimized.
This comment has been minimized.
lock
bot
commented
Mar 22, 2019
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
fabxc commentedFeb 8, 2017
The tool we use to embed assets into the binary has issues around timestamping. That's the reason why we don't always trigger it on build (commit pollution), which often leads to it being forgotten on actual changes.
https://github.com/rakyll/statik might be an alternative.