Openssl proper support for multiple defines

[klausholstjacobsen]

openssl/3.x.x

With current recipe multiple defines in AutotoolsToolchain results in this config:

openssl/3.0.13: my %targets = (
    "conan-Release-Linux-x86_64-gcc-11" => {
        inherit_from => [ "linux-x86_64" ],
        cflags => add("-m64 -fPIC -O3"),
        cxxflags => add("-m64 -fPIC -O3"),
        defines => add("NDEBUG MYDEF1 MYDEF2"),
        lflags => add("-m64"),
    },
);

Resulting in the gcc compile command ...-DNDEBUG MYDEF1 MYDEF2... (note the missing '-D') which produces a compile error.

This PR fixes that issue producing this working config:

openssl/3.0.13: my %targets = (
    "conan-Release-Linux-x86_64-gcc-11" => {
        inherit_from => [ "linux-x86_64" ],
        cflags => add("-m64 -fPIC -O3"),
        cxxflags => add("-m64 -fPIC -O3"),
        defines => add("NDEBUG"),
        defines => add("MYDEF1"),
        defines => add("MYDEF2"),
        lflags => add("-m64"),
    },
);

• I've read the contributing guidelines.
• I've used a recent Conan client version close to the currently deployed.
• I've tried at least one configuration locally with the conan-center hook activated.

[github-actions]

🤖 Beep Boop! This pull request is making changes to 'recipes/openssl//'.

👋 @Hopobcn @Croydon you might be interested. 😉

[ghost]

I detected other pull requests that are modifying openssl/3.x.x recipe:

• openssl/3.2.1: Patch for MD5 cert names on Android. #23562
• openssl: add version 3.3.0 #23469
• openssl: make things a bit more generic #23304

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

[Croydon]

@klausholstjacobsen Have a look at #22662

[conan-center-bot]

Conan v1 pipeline ❌

Failure in build 2 (fe6ad874ea65344eced46de69e7a0c92448a5ad6):

An unexpected error happened and has been reported. Help is on its way! 🏇

---

Conan v2 pipeline ✔️

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

All green in build 2 (fe6ad874ea65344eced46de69e7a0c92448a5ad6):

• openssl/3.2.0:
  All packages built successfully! (All logs)

• openssl/3.1.5:
  All packages built successfully! (All logs)

• openssl/3.2.1:
  All packages built successfully! (All logs)

• openssl/3.1.4:
  All packages built successfully! (All logs)

• openssl/3.1.3:
  All packages built successfully! (All logs)

• openssl/3.0.13:
  All packages built successfully! (All logs)

• openssl/3.0.12:
  All packages built successfully! (All logs)

[Croydon]

This should be fixed via #22662

[klausholstjacobsen]

Fixed via #22662