Gregtech++ [GT++] [GTplusplus]

Gregtech++ [GT++] [GTplusplus]

94.1k Downloads

PRs with GT5U

mura3277 opened this issue · 14 comments

commented

Things like the Energy Buffers and Industrial Coke Oven would make really nice additions on Blood's fork.

Have you thought about submitting PRs?

commented

I have, but decided keeping it as a separate mod is a better idea,
Between GT5.08, GT5.09 and my own Fork, I can maintain cross compat with all 3.

commented

@Muramasa- , Give me some time to think about a way for modular addition.
If we added it to the master branch, worlds with my mod in them would have a double of entities.

We'll have to check against particular entities already existing, since just adding them with the same ID's will cause a load failure.

commented

@Muramasa- what's your best form of contact?

commented

@ me on here or PM me on the IC2 forums (Muramasa on there). I get email notifications via both.

commented

@Muramasa-, I Probably pm you then, Would make a discussion easier.. Unless you wanted to join me for a chat on IRC?

I'm in #gregtech and #gt-dev on Epsernet 24/7

commented

Sure, I'm on the IRC now (Name is Muramasa).

commented

So, I spoke with @OneEyeMaker, His latest PR adds a new black casing class, so If that's accepted I'll use that to integrate the Coke Oven.

If I add in the Power Buffers or the Coke Oven, I'd like them to Check for the presence of my own mod first and not add them if it's present. This makes compatibility easiest as there are many worlds with these already in them. No point re-adding content already generated by GT++ (My mod)

commented

I assume you can just fork his repo, There's quite a few things he hasn't implemented from Experimental so it me be easy to work that way instead of the other. I am lacking a way to translate his comments into any decent readable form so I couldn't tell you from memory what is missing sorry. (Dammit Google and other translator tools)

commented

Yep, that seems like the best idea to keep compatibility with all versions.

So I seen you talking on IRC about how @Bogdan-G has made some really nice performance improvements. It would be really nice to have that as well. (When ever you guys get around to it of course.)

commented

It would be really nice to have that as well. (When ever you guys get around to it of course.)

That's more of something to ask him, being Russian and some kind of pro Java programmer, most of his commits make little sense. If you look at his Repo, you can see the optimizations he's going for and I doubt he has much interest making any PRs, considering his fork has been separated to it's own branch.

commented

Ah ok I see. May I ask when you next talk to him to see if its ok that I commit his improvements to the master branch on his behalf?

commented

🐹

commented

Hello @Bogdan-G!

Can I Pull Request Bogdan-G/GT5-Unofficial@9d05c48 to https://github.com/Blood-Asp/GT5-Unofficial and other performance improvements?

commented

@Muramasa- yes, my changes base on GT5U license(or LGPL). If you have questions about my changes, then you ask.

PS hmm i will be necessary to add in the my repo which license is used to avoid confusion.
PPS Bogdan-G/GT5-Unofficial@d415df7...master
PPPS minor info: im used compile openjdk7u101, git-scm bash-console 2.8+, gradle 2.13+