Latest Posts

Topic: __skip_if_exists ?

kaputtnik
Avatar
Topic Opener
Joined: 2013-02-18, 20:48
Posts: 2433
OS: Archlinux
Version: current master
Ranking
One Elder of Players
Location: Germany
Posted at: 2021-05-30, 18:48

Just noticed that the foreign_planet add-on uses "__skip_if_exists"

I have tried to find some source about this, but i haven't using grep. So what is the purpose of this and where is it documented?

Do we need a new forum for Add-On's below Widelands Development ?


Fight simulator for Widelands:
https://wide-fighter.netlify.app/

Top Quote
Nordfriese
Avatar
Joined: 2017-01-17, 18:07
Posts: 1928
OS: Debian Testing
Version: Latest master
Ranking
One Elder of Players
Location: 0x55555d3a34c0
Posted at: 2021-05-30, 18:56

Directives like this one will be introduced in https://github.com/widelands/widelands/pull/4760 . In master it's still ignored at the moment. The purpose is to handle the case that two add-ons define a unit with the same name. So all authors of ‘World’ and ‘Tribes’ add-ons will need to use it in the future.

(Currently in master, the last add-on in the list takes priority in this case. Whether this is actually safe is heavily add-on dependent; these directives are the only truly safe solution approach. I'm intending to add more, optional such directives in the future to give add-on designers even more control about the way their add-on interacts with other add-ons.)

Do we need a new forum for Add-On's below Widelands Development ?

+1 face-smile.png
But IMHO this would fit better in the Player Forums category, since such a forum will probably chiefly be used to discuss the add-ons themselves and not just their development. We already have the development forum for the various aspects of all the implementation details.

Edited: 2021-05-30, 18:58

Top Quote
kaputtnik
Avatar
Topic Opener
Joined: 2013-02-18, 20:48
Posts: 2433
OS: Archlinux
Version: current master
Ranking
One Elder of Players
Location: Germany
Posted at: 2021-05-30, 20:29

Thanks for the explanation face-smile.png

New Forum is available.


Fight simulator for Widelands:
https://wide-fighter.netlify.app/

Top Quote
Nordfriese
Avatar
Joined: 2017-01-17, 18:07
Posts: 1928
OS: Debian Testing
Version: Latest master
Ranking
One Elder of Players
Location: 0x55555d3a34c0
Posted at: 2021-05-30, 20:44

Great face-smile.png Just one nit, can you please remove the apostrophe in the forum name? "Add-On's" is a genitive, you can use it to say "The add-on's author" or "The add-on's contents", but a plural form ("the Add-Ons") can not contain one.

For a description, how about "The place to discuss all kinds of user-made content."

IMHO the add-on submission tread should also be moved to the new forum.


Top Quote