Fix: exceptions during build of bitserializer

[jrosiek]

Summary

Changes to recipe: bitserializer/[>=0.50]

Motivation

The build fails when two variants of the package are build on the same machine (with different with_rapidaml option value).

Details

(Apparently) The same source location is used by conan to build different variants of package. In effect, the same file is patched more than once, what causes an exception.

---

• Read the contributing guidelines
• Checked that this PR is not a duplicate: list of PRs by recipe
• Tested locally with at least one configuration using a recent version of Conan

[jcar87]

Hi @jrosiek, thanks for your contribution.

Could you provide more details as to which exceptions you get and how to reproduce the errors on our end? Thanks :)

[jrosiek]

Here you go:

conan remove 'bitserializer/*'
conan install --requires=bitserializer/0.70 -o '&:with_csv=True' -o '&:with_rapidyaml=True' --build=missing
conan install --requires=bitserializer/0.70 -o '&:with_csv=True' -o '&:with_rapidyaml=False' --build=missing

The with_csv option is required for the recipe to generate unique package_id for each install call; otherwise the configuration is treated as header-only and the second call would do nothing.

[conan-center-bot]

Conan v1 pipeline ✔️

Warning

Conan Center will stop receiving updates for Conan 1.x packages soon - please see announcement.

All green in build 2 (8e51b7ad9a6b4efcf9bfa7ce77e86349914602f6):

• bitserializer/0.65:
  Built 5 packages out of 11 (All logs)

• bitserializer/0.70:
  Built 5 packages out of 11 (All logs)

• bitserializer/0.50:
  Built 5 packages out of 11 (All logs)

• bitserializer/0.44:
  Built 5 packages out of 11 (All logs)

• bitserializer/0.10:
  Built 5 packages out of 11 (All logs)

---

Conan v2 pipeline ✔️

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

All green in build 2 (8e51b7ad9a6b4efcf9bfa7ce77e86349914602f6):

• bitserializer/0.70:
  Built 4 packages out of 5 (All logs)

• bitserializer/0.65:
  Built 4 packages out of 5 (All logs)

• bitserializer/0.50:
  Built 4 packages out of 5 (All logs)

• bitserializer/0.10:
  Built 4 packages out of 5 (All logs)

• bitserializer/0.44:
  Built 4 packages out of 5 (All logs)

[AbrilRBS]

@jrosiek thanks a lot for your PR. Could you try checking if the current changes also solve your issue? Thanks!

[jrosiek]

Yes, the change fixes the issue on my side. Thanks.

[jcar87]

I'm just double checking the very first commit of this recipe that contained the no_copy_source: d4f8bc2

and no context is provided to justify it - it is useful in some cases, but don't think it is necessary here