git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] ci: run `make sparse` as a GitHub workflow

From
Ramsay Jones <ramsay@ramsayjones.plus.com>
Date
Jul 16, 2021, 01:37 UTC
Message-ID
<aeecbcf6-eab8-54d4-9654-b5b2e1e1ea23@ramsayjones.plus.com>
In-Reply-To
<nycvar.QRO.7.76.6.2107141212140.76@tvgsbejvaqbjf.bet>
On 14/07/2021 11:13, Johannes Schindelin wrote:
Show 46 quoted lines
> Hi again,
> 
> On Wed, 14 Jul 2021, Johannes Schindelin wrote:
> 
>> On Tue, 13 Jul 2021, Philippe Blain wrote:
>>
>>> Le 2021-07-13 à 07:51, Johannes Schindelin via GitGitGadget a écrit :
>>>> From: Johannes Schindelin <johannes.schindelin@gmx.de>
>>>>
>>>> [...]
>>>> +    - name: Download the `sparse` package
>>>> +      uses: git-for-windows/get-azure-pipelines-artifact@v0
>>>> +      with:
>>>> +        repository: git/git
>>>> +        definitionId: 10
>>>> +        artifact: sparse-20.04
>>>> +    - name: Install the `sparse` package
>>>> +      run: sudo dpkg -i sparse-20.04/sparse_*.deb
>>>
>>> Out of curiosity, why is this necessary (as opposed to using
>>> the Ubuntu package, i.e. 'sudo apt-get -q -y sparse') ?
>>
>> This is actually a historical curiosity: years ago, I created an Azure
>> Pipeline that builds the `sparse` Debian package for the specific purpose
>> of using it in our CI builds (if you care to look at the issue 345 I
>> linked above, you will see how long ago that idea was in the making). Now,
>> the historical curiosity is that back then, there was no current `sparse`
>> package available for Ubuntu, and Ramsay mentioned that a newer version
>> would be required to run `make sparse`.
>>
>> And when I implemented this patch yesterday, I did not even question this,
>> I was just happy that I had come up with the GitHub Action
>> `get-azure-pipelines-artifact` (to help with the `vcpkg` part of our CI
>> builds).
>>
>> I was already writing a detailed paragraph in the commit message to
>> explain all that when it occurred to me that two years might make a big
>> difference and an up to date `sparse` might be available. And lo and
>> behold, this is the case!
>>
>> Therefore, v2 will no longer jump through that hoop.
> 
> *Never mind*
> 
> I made a mistake in testing, and Ubuntu-20.04's `sparse` package _is too
> old_. So I will reintroduce that hoop even before sending v2.

Yes, last time I looked, 20.04's version of sparse was 0.6.1-2build1, 20.10 had 0.6.2-2 (both of which are too old), but 21.04, along with Debian Testing, had 0.6.3-1 which would work fine.

[I am currently running v0.6.3-341-g8af24329]
So, maybe the next Ubuntu LTS, ... :-D

ATB, Ramsay Jones

Previous: Johannes SchindelinNext: Junio C Hamano
Message 7 of 22 in “ci: run `make sparse` as a GitHub workflow”
  1. ci: run `make sparse` as a GitHub workflowJohannes Schindelin via GitGitGadget, Jul 13, 2021
  2. Đoàn Trần Công DanhJul 13, 2021
  3. Johannes SchindelinJul 14, 2021
  4. Philippe BlainJul 13, 2021
  5. Johannes SchindelinJul 14, 2021
  6. Johannes SchindelinJul 14, 2021
  7. Ramsay JonesJul 16, 2021
  8. Junio C HamanoJul 13, 2021
  9. Johannes SchindelinJul 14, 2021
  10. Junio C HamanoJul 14, 2021
  11. Johannes SchindelinJul 14, 2021
  12. Junio C HamanoJul 14, 2021
  13. Johannes SchindelinJul 14, 2021
  14. Junio C HamanoJul 14, 2021
  15. Johannes SchindelinJul 16, 2021
  16. Junio C HamanoJul 16, 2021
  17. ci: run `make sparse` as part of the GitHub workflowJohannes Schindelin via GitGitGadget, Jul 14, 2021
  18. ci: run "apt-get update" before "apt-get install"Jeff King, Jul 26, 2021
  19. Jeff KingJul 26, 2021
  20. Junio C HamanoJul 26, 2021
  21. ci: run `make sparse` as part of the GitHub workflowJohannes Schindelin via GitGitGadget, Jul 26, 2021
  22. Junio C HamanoJul 26, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.