Add new API additions from Glowkit to Spigot's Bukkit?

[deathcap]

Since forking from the common ancestor https://github.com/Bukkit/Bukkit circa Minecraft 1.7, GlowstoneMC's updated Bukkit (Glowkit) and SpigotMC's updated Bukkit have diverged for new API added to support 1.8.x. There is some overlap, but SpigotMC has API that GlowstoneMC does not, and GlowstoneMC has also has API that SpigotMC does not have either. So I think it would be worth considering whether some of GlowstoneMC's API enhancements could be submitted to the SpigotMC team for inclusion in their updated version of Bukkit, towards the goal of a unified common API.

This would be advantageous for Glowstone since the new APIs would gain more widespread usage, and also for plugin developers since they would be able to target both Glowstone and Spigot with the same API methods, instead of having to code separately for Glowstone and Spigot, when using >1.7 features. A win-win scenario.

SpigotMC could also have feedback before merging which could be used to improve the specific APIs in both places, providing a unified interface to both implementations. Details on their submission process at: https://hub.spigotmc.org - the readme for their specific "Bukkit API" project https://hub.spigotmc.org/stash/projects/SPIGOT/repos/bukkit/browse says "The development team is very open to both bug and feature requests / suggestions". Worth a shot?

Here's a short list of non-conflicting (to my knowledge) APIs developed by GlowstoneMC, I believe would make sense to approach for submission to SpigotMC:

• GlowstoneMC/Glowkit@16cb5d6 Added cause-specific damage methods to Damageable. by @PaulBGD in https://github.com/GlowstoneMC/Glowkit/pull/33
• GlowstoneMC/Glowkit@6ecc135 Updated resource packs for 1.8 changes, with resource pack event. by @Aaron1011 in https://github.com/GlowstoneMC/Glowkit/pull/49
• GlowstoneMC/Glowkit@e2d8849 Added MaterialData for Prismarine, Sand, and Sponge. by @SpaceManiac
• GlowstoneMC/Glowkit@53d3e69 Added missing effects from Minecraft 1.7. by @socram8888 in Add Villager Career API Glowkit#13
• Tonodus@f9a7fba InventoryView.Property. for enchantments in 1.8 (WIP) by @Tonodus in https://github.com/GlowstoneMC/Glowkit/pull/63
• GlowstoneMC/Glowkit@267a795 Added MaterialData for Dirt, Stone, and DoublePlant. by @jimmikaelkael in https://github.com/GlowstoneMC/Glowkit/pull/29 https://github.com/GlowstoneMC/Glowkit/pull/30 Update to 1.17 Glowkit#27

Tagging the original authors since it is probably best if they submit their code to SpigotMC if they choose to do so.

This is only an initial list, might have more to add as I progress through the changes.

Note this task issue is only concerning GlowstoneMC → SpigotMC, the reverse direction SpigotMC → GlowstoneMC has a separate issue: GlowstoneMC/Glowkit#18

[gdude2002]

They expect you to fill out this CLA.