Is it possible to concatenate a bunch of file dige...
# pluginsr
ripe-gigabyte-88964
12/20/2023, 4:25 PMIs it possible to concatenate a bunch of file digests together, or is there a built in method/rule to convert bytes into
FileDigestripe-gigabyte-88964
12/20/2023, 4:26 PMContext is these custom acceptance tests I'm writing run a couple of different processes and I just want to concatenate the stdout and stderrs together for the test result.
b
bitter-ability-32190
12/20/2023, 4:31 PMYes: https://github.com/pantsbuild/pants/blob/78775b9716f93609a599d9efd17e169e574f17f2/src/python/pants/engine/fs.py#L22C1-L22C78
(which is from https://github.com/pantsbuild/pants/blob/78775b9716f93609a599d9efd17e169e574f17f2/src/python/pants/engine/internals/native_engine.pyi#L415)
And Yes: https://github.com/pantsbuild/pants/blob/78775b9716f93609a599d9efd17e169e574f17f2/src/python/pants/engine/fs.py#L45C7-L45C18
r
ripe-gigabyte-88964
12/20/2023, 4:32 PMoh I didn't realize merge digests would work for
FileDigestb
bitter-ability-32190
12/20/2023, 4:33 PMI don't think that'd work, the types don't match.
bitter-ability-32190
12/20/2023, 4:34 PMA
FileDigestr
ripe-gigabyte-88964
12/20/2023, 4:37 PMCan I just create a
FileDigestb
bitter-ability-32190
12/20/2023, 4:39 PMI think you'd have to do:
• CreateDigest([FileContent(,,,)) -> Digest
• Digest -> DigestEntries
• digest_entries[0].file_digest
But that seems like maybe we're coming at the problem wrong? Why do you just want the file digest?
r
ripe-gigabyte-88964
12/20/2023, 4:41 PMTo pass to test result, since I want to concatenate different the stdouts and stderrs of the various processes that are running together https://github.com/pantsbuild/pants/blob/78775b9716f93609a599d9efd17e169e574f17f2/src/python/pants/core/goals/test.py#L79-L82
b
bitter-ability-32190
12/20/2023, 4:41 PMAh I see
bitter-ability-32190
12/20/2023, 4:42 PMYeah, I think IIUC that'd be the only way. You can one-shot this for both stderr and stdout by
CreateDigestFileContentr
ripe-gigabyte-88964
12/20/2023, 4:43 PMCool, will do that then. Thanks!
b
bitter-ability-32190
12/20/2023, 4:43 PMI wonder why that field is there though. Seems out of place to me 😅
2 Views