westes / flex

The Fast Lexical Analyzer - scanner generator for lexing in C and C++
Other
3.55k stars 529 forks source link

add apt-get update to ci #595

Closed vmeshche closed 11 months ago

vmeshche commented 11 months ago

apt-get install failed with E: Failed to fetch. Looks like some old deb staff in cache and we need run apt-get update before setapping env.

Mightyjo commented 11 months ago

Looks like github fixed this by updating their runner images.

westes commented 11 months ago

In general, this approach makes builds less reproduceable, not more. This also makes it harder to tell what the underlying OS contains. If we're concerned that particular versions of packages should or should not be present, then we should simply specify what we need rather than adding a step that makes the process take longer and be less determined.