On 12/10/2013 10:14, Christoph Hellwig wrote: > attempts to bypas the license we put on the code and vilationg our copyrights. I seriously appreciate the clear description of what you were thinking when reading my message. I will also be clear: My intention is to fully respect developer's decisions to protect APIs and (obviously) respect laws. Original author's decision in this regard is clear: posix_acl_init,posix_acl_alloc,posix_acl_valid,posix_acl_equiv_mode,posix_acl_from_mode,posix_acl_chmod, posix_acl_create are explicitly exported for modules with _ANY_ license. Therefore I'm authorized to use them in a CDDL-licensed module. There is one function, posix_acl_release, whose use is clearly related to posix_acl_alloc, which does not have a clearly expressed export policy, as it is an inline function written in a header file. Through multiple macro expansions, it calls a GPL-only symbol. Being an inline function, the linker won't allow that. Do you agree with me that this is a quite weak to express the decision to not allow the use of Posix ACL's functions in non-GPL modules? This is the reason why I tought this might be an oversight. If original authors think that Posix ACL APIs should not be used from non-GPL modules, I suggest you to export also: posix_acl_init posix_acl_alloc posix_acl_valid posix_acl_equiv_mode posix_acl_from_mode posix_acl_chmod posix_acl_create as GPL-only symbols. Currently, they are available to modules with any license. Your intent of protecting Filesystem APIs will be clear to anyone and will be respected without any more questions on this mailing list. This is an inconsistency that IMHO should be fixed. Fixing it in a more liberal way or in a more strict way is up to the original authors. Sorry for the long message, but I felt misunderstood and wanted to make my point clear. Regards, Massimo Maggi -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
