Fix caching on databroker build

[SebastianSchildt]

There was a typo that prevents full caching of build artifacts for databroker build (CLI was ok already). Fixed now.

Example build with no cache (so probably also the build here, as changing cache contents invalidates cache)

Subsequent build satisfying cache requirements

(see here https://github.com/boschglobal/kuksa-databroker/actions?query=event%3Apull_request+branch%3Atest%2Fcaching for various test builds)

I also checked where time is mainly used now

• 20+ sec is always used pulling the cross images (crossb uilds using docker). the cross cargo package is cached and not reinstalled but the docker images are not. I do not intend to look into this/fix this in this PR. Just a hint, if somebody feels bored. I am however not sure if there is even a feasible solution as those images are around 2 GB each and I think there are strict limitations on how much cache can be used before things get evicted)
• Another large chunk is basically the "linking" so I think, this has to be accepted.

[argerus]

This seems to fix caching for the build step.

The overall build is still slower than it was since caching of the unit test step was also broken in the original PR.

I suppose always having the target dir under target/ would make it easier to get this right, i.e. using --target-dir "target/cross-$target" in order to work around the issue with building for different systems under cross.

[SebastianSchildt]

Fixed. I choose to fix the cache definition in workflow instead of changing dirs in script. (I was thinking if somebody during dev is using plain "cargo build/run" and also in between places with the build scripts, the structure inside the default target folder might seem confusing. Also... easier/faster to test for me just now)