search

INPStarfall / Starfall

Starfall Processor for Garry's Mod + Wiremod
http://www.wiremod.com/forum/developers-showcase/22739-starfall-processor.html
Other
17 stars 15 forks source link

Added constraint library #402

Open davidsonbr opened 9 years ago

davidsonbr commented 9 years ago

Not sure if I should just make these methods for the entity type. LMK

awilliamson commented 9 years ago

Although the constraint library requires Entity as a dependency doesn't mean it should be included as such. It's namespaced separately and the functions used are separate. They just take input of entities.

awilliamson commented 9 years ago

@thegrb93 It's not difficult to make this library. Most of it is boilerplate anyway. A look at the docs for the constraint library on the GMod wiki gives you everything you need.

I reference: https://github.com/INPStarfall/Starfall/pull/328#issuecomment-113629783 And for the sake of completeness

It's funny you think reusing open code [...] is considered plagiarism. I copy pasted it to save time and got the task done.

You can't have it both ways. If you think that @EpicG, for whatever reason, copied your library rather than making it from scratch prove so. Prove the original content, prove it wasn't open source or that the license prohibited redistribution or derivative works. As far as I can see, this isn't a 'difficult' addition.

As for:

I made it and then I closed the pr 3 days later because the contributors are too lazy to merge

In the real world PRs don't get resolved within 3 days usually, it's actually rare for that to happen unless it's a small 1 hunk change.

Xandaros commented 9 years ago

@thegrb93 Well, it took me a moment to see what looks copied and what doesn't, but I finally found a function: getTable. With one minor difference, that being that EpicG's version actually works, they are identical.

The funny thing is, though... his version was made in multiple commits, wheras yours is just a single one. Why go through the trouble of splitting it up, especially when they aren't even strictly about that function, but rather on the library as a whole? You obviously created it with a single commit, he then chopped off the permissions only to reintegrate them exactly as you have later, together with permissions for the rest of the library? I think not.

Your version: His version:

As far as I'm concerned, you stole from him!

Oh yeah, his commits also pre-date yours. Just saying.

awilliamson commented 9 years ago

@thegrb93 In light of this evidence, I suggest you think very carefully next time you try to claim that people are stealing from you when in fact you have stolen from them. Now I've seen that getTable function ( which is very specific in implementation ), it appears to be copied. It's like how you catch people copying on tests, because they get the same section wrong in the exact same, odd, way. I can't believe you have the audacity to accuse @EpicG of stealing when you have done so yourself, as evidenced by the above.

This is the very exact reason you were initially banned from contributing to this organisational repo ( Which you should still be banned from? ). It is apparent that you haven't learnt anything in that time on how to function as a member of the open source community or to get along with your fellow developers.

awilliamson commented 8 years ago

@EpicG I remember you saying this was still a bit WIP. Any updates in regards to this?