Page MenuHomeSoftware Heritage

opam: Initialize opam root directory outside the constructor
ClosedPublic

Authored by ardumont on Sep 21 2021, 7:36 PM.

Details

Summary

If it's required at all, this will use the network to fetch and install it.
This should be done outside the constructor.

Related to T3590

Test Plan

tox

Diff Detail

Repository
rDLDBASE Generic VCS/Package Loader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build is green

Patch application report for D6317 (id=22951)

Rebasing onto fabd950d36...

Current branch diff-target is up to date.
Changes applied before test
commit f4556e0113fae658555b0c6f7296ac98c310c971
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Sep 21 19:15:31 2021 +0200

    opam: Initialize opam root directory outside the constructor
    
    If it's required at all, this will use the network to fetch and install it.
    This should be done outside the constructor.
    
    Related to T3590

See https://jenkins.softwareheritage.org/job/DLDBASE/job/tests-on-diff/560/ for more details.

This revision is now accepted and ready to land.Sep 22 2021, 10:38 AM