On Freitag, 15. Dezember 2017 11:32:05 CET Sven Eckelmann wrote:
On Mittwoch, 6. Dezember 2017 11:58:14 CET Willem de Bruijn wrote: [...]
MAINTAINERS | 2 +- include/uapi/linux/{batman_adv.h => batadv_genl.h} | 6 +++---
This and the previous patch changes uapi. That might break userspace applications that rely on it.
I am not aware of any application because all (alfred, batctl and some gluon integration) of them currently ship their own copy because distribution didn't catch up. And this is also the reason why I want to do it now - not later.
That assumes that you know all applications, including those not publicly available. It may be true in this instance, but it is not possible to be certain.
I've just talked with Simon. Because you have a problem with these two changes, he suggested that I should drop these two patches and merge packet.h with the uapi batadv genl header batman_adv.h
No, this is also bad because batman_adv.h is MIT license and packet.h is GPL-2. So what other name would you suggest for packet.h? batman_adv_packet.h?
Kind regards, Sven