Skip to content

Don't install apt-fast from source#96

Merged
awalsh128 merged 1 commit intoawalsh128:masterfrom
pascallj:apt-fast
Mar 6, 2023
Merged

Don't install apt-fast from source#96
awalsh128 merged 1 commit intoawalsh128:masterfrom
pascallj:apt-fast

Conversation

@pascallj
Copy link
Copy Markdown
Contributor

There is no reason to install apt-fast from source as it is already included in all of the Ubuntu runner images.

@awalsh128 awalsh128 merged commit 854bb53 into awalsh128:master Mar 6, 2023
@awalsh128
Copy link
Copy Markdown
Owner

Thank you!

@pascallj pascallj deleted the apt-fast branch March 6, 2023 15:00
@mstruebing
Copy link
Copy Markdown

Hey there, we were just testing your action but we are using custom runners with custom runner images.
That being said, would you be open to an extra argument to that action that would install apt-fast from the source if present as our runner images don't have these preinstalled?

If you like that idea, I would try to make a PR with that change.

@pascallj
Copy link
Copy Markdown
Contributor Author

pascallj commented Mar 7, 2023

You're right, custom runners are also a thing. There is no need to implement an extra argument for that. We can easily detect if apt-fast is present and executable. I've created PR #97 to address this.

awalsh128 pushed a commit that referenced this pull request Mar 8, 2023
* Revert "Don't install apt-fast from source (#96)"

This reverts commit 854bb53.

* Only install apt-fast from source if not present
awalsh128 added a commit that referenced this pull request Mar 24, 2023
* More robust checking of age of apt cache (#90)

* Don't install apt-fast from source (#96)

* Clean cache directory after use (#95)

* Only install apt-fast from source if not present (#97)

* Revert "Don't install apt-fast from source (#96)"

This reverts commit 854bb53.

* Only install apt-fast from source if not present

---------

Co-authored-by: John Hughes <jondo2010@gmail.com>
Co-authored-by: Pascal Roeleven <dev@pascalroeleven.nl>
awalsh128 added a commit that referenced this pull request Mar 24, 2023
* More robust checking of age of apt cache (#90)

* Don't install apt-fast from source (#96)

* Clean cache directory after use (#95)

* Only install apt-fast from source if not present (#97)

* Revert "Don't install apt-fast from source (#96)"

This reverts commit 854bb53.

* Only install apt-fast from source if not present

---------

Co-authored-by: John Hughes <jondo2010@gmail.com>
Co-authored-by: Pascal Roeleven <dev@pascalroeleven.nl>
awalsh128 added a commit that referenced this pull request Mar 24, 2023
* More robust checking of age of apt cache (#90)

* Don't install apt-fast from source (#96)

* Clean cache directory after use (#95)

* Only install apt-fast from source if not present (#97)

* Revert "Don't install apt-fast from source (#96)"

This reverts commit 854bb53.

* Only install apt-fast from source if not present

---------

Co-authored-by: John Hughes <jondo2010@gmail.com>
Co-authored-by: Pascal Roeleven <dev@pascalroeleven.nl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants