Based on a quick look, much of it does in fact appear thread-safe. There are
grues in the darker corners though :eyes:
One certain problem is the global macro space, readFilesManifest() does
push/pop on %license which is probably harmless, but further down the call
chain there's readManifest() and specExpand() which do push/pop on current file
name and line number, which is not. So at the very least the files manifest
parsing would have to be lifted to a pre-parallel section. I'd be nothing short
of shocked if this was the only gotcha there is, as processPackageFiles() ends
up covering a scary amount of code.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/1185#issuecomment-617127136
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint
