Jammy support

Fix compilation on Ubuntu 22.04

assigned to @hahiho

requested review from @dreher

What's the exact issue here?

requested review from @fratty

With the new default compiler, aggregate initialization is not possible if these constructors are defined. See also here. Somewhere the struct was initialized with designated initializers and since GCC 9 this is prohibited with user-declared constructors.

But maybe removing the constructor is not the best solution and we should get rid of the aggregate initialization instead?

But maybe removing the constructor is not the best solution and we should get rid of the aggregate initialization instead?

I think I'd prefer that (since there is a custom constructor). What do others think?

Do we know what the constructor is doing? If its just the initialization of members we can also use the designated initializer lists. If has a body we should use the constructor I would say.

PS: If its because of the SkillDescription: IMHO its fine to remove the constructor. I think the only reason I used it was the default argument or so which can also be put to the variable declaration. Don't really remember.

        SkillDescription::SkillDescription(const std::string& name, const std::string& desc, const std::vector<std::string>& r, long timeoutMs, const aron::type::ObjectPtr& type, const aron::data::DictPtr& def) :
            skillName(name), description(desc), robots(r), timeoutMs(timeoutMs), acceptedType(type), defaultParams(def)
        {
        }

Looks like pretty simple initialization

added 115 commits

• 455f4941...435b9e07 - 114 commits from branch master
• a2cd4d1f - Merge remote-tracking branch 'origin/master' into ubuntu_2204

Compare with previous version

added 1 commit

• d85d615d - Fix compile issues under U22

Compare with previous version

mentioned in merge request sw/armarx/meta/axii!195 (merged)

Can this now be merged or is action needed? Unclear. @peller @kartmann?

There are just 2 changes which look harmless, so I think we can merge it

merged

mentioned in commit e7d800fb