> I think
>
> add_definitions(-DGETTEXT_PACKAGE=\"${GETTEXT_PACKAGE}\"
> -DGNOME_DESKTOP_USE_UNSTABLE_API)
>
> should become
>
> add_definitions(-DGETTEXT_PACKAGE=\"${GETTEXT_PACKAGE}\")
> add_definitions(-DGNOME_DESKTOP_USE_UNSTABLE_API)
>
> Maybe with some comments above the lines
This is not needed. They would have called it "add_definition" then ;)
> I think (-DGETTEXT_ PACKAGE= \"${GETTEXT_ PACKAGE} \" DESKTOP_ USE_UNSTABLE_ API) (-DGETTEXT_ PACKAGE= \"${GETTEXT_ PACKAGE} \") (-DGNOME_ DESKTOP_ USE_UNSTABLE_ API)
>
> add_definitions
> -DGNOME_
>
> should become
>
> add_definitions
> add_definitions
>
> Maybe with some comments above the lines
This is not needed. They would have called it "add_definition" then ;)