commit 28b44a8fb4846b2f710081a04ce1851c4ac43b99
author Václav Kubernát <kubernat@cesnet.cz> Tue Apr 05 09:54:43 2022 +0200
committer Václav Kubernát <kubernat@cesnet.cz> Tue Apr 05 10:04:01 2022 +0200
tree 35b7daf8d9d0b4401c9b07249c185131aa07956a
parent 218ee7bcbef1c066a2be56f087ab8c824b529188

Fix PathParser attribute_type

This took me a while... For some reason, Spirit does not really NEED
this `attribute_type` thing to work properly. Or maybe it does, but
doesn't care if it's invalid, because obviously, `::type` has to be
used, otherwise the using is just the ModeToAttribute type.

Anyway, fixing this revealed another bug on the code: until now, the
getPath parser used decltype(get_::m_path) as the attribute, which is
`boost::optional<boost::variant<dataPath_, module_>>. However, this is
not the correct attribute, because the getPath parser can only parse a
dataPath_, or a module_. The "optional wrapping" is done via the
optional parser directly in the rule for the get command.

The reason I found all of this is that I wanted add an optional argument
for `get` and I kept getting errors for "can't assign
boost::optional<boost::variant<...>> to boost::variant<...>".

Change-Id: I76ff8d3f201b58bb83b68c428f6b95f5c87f3a75