digital-asset/daml
1
1
Fork 0
mirror of https://github.com/digital-asset/daml.git synced 2024-09-20 01:07:18 +03:00
Code Issues Projects Releases Wiki Activity

Fix DD test case 'Cross Daml-LF version with double data-dependency' (#15371)

Browse Source 
The problem is that `damlcForTarget tools depLfVer` could return an older damlc
than needed for `targetLfVer`.

The reason this didn't fail before is that the legacy damlc supports up to
LF=1.14, and `lfVersionTestPairs` never had `depLfVer < 1.14 && targetLfVer > 1.14`

changelog_begin
changelog_end
This commit is contained in:
Moisés Ackerman 2022-10-27 14:03:03 +02:00 committed by GitHub
parent e8c7e29ba2
commit d2f824fcc2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
compiler/damlc/tests/src/DA/Test/DataDependencies.hs
View File

@ -343,7 +343,7 @@ tests tools = testGroup "Data Dependencies" $
, " - " <> show oldProjDar
, " - " <> show (proja </> "proja.dar")
]
callProcessSilent (damlcForTarget tools depLfVer)
callProcessSilent damlc
["build"
, "--project-root", projb
, "--target", LF.renderVersion targetLfVer