Closed ProgrammerDan closed 8 years ago
Can one of the admins verify this patch? Type 'ok to test' to test.
ok to test
@rourke750 or @Maxopoly can I get some eyes on this and a test build? The pom dependencies are a frickin' mess in this plugin, so contrary my usual I'm submitting this pull blind.
ok to test
oooor triggers aren't working, sigh.
@Maxopoly do you have it setup to build locally?
@rourke750 any chance my account on the build server could get admin so I could fix some of these build jobs? Been using Jenkins for years, I could be an asset.
What's your account.
ok to test
It's ProgrammerDan -- i'm nothing if not consistent :D
Jar built
@ProgrammerDan you have jenkins perms now.
Looks like you have it pretty segmented, not seeing any configuration controls on any active projects. Do see some new options on prior builds (redeployment options, etc.)
give the man full access.
hell he can have ssh into the machine if he wants.
@ProgrammerDan try now, i had missed some.
Alright, build is live on Civtest Pepper.
had a few players test it for me on civtest, works as desired:
@ttk2 @rourke750 @erocs @drjawa Any objections / discussion, or safe to merge?
merge and I will stage immediately thereafter.
On Thu, Nov 12, 2015 at 10:24 AM, Daniel Boston notifications@github.com wrote:
@ttk2 https://github.com/ttk2 @rourke750 https://github.com/rourke750 @erocs https://github.com/erocs @drjawa https://github.com/drjawa Any objections / discussion, or safe to merge?
— Reply to this email directly or view it on GitHub https://github.com/Civcraft/PrisonPearl/pull/75#issuecomment-156155093.
Give me a min to look at it.
Ah woops. Feel free to rollback if you see something undesired.
Youre good you did exactly what I was checking for. Just making sure we are up keeping the softdepdency.
Yeah, definitely.
Only thing I didn't do is up the version number on the plugin. If you want I can do that real quick-like -- always forget, shame on me.
Rourke never does. No worries, you're on par.
Not sure this needed a <24hr fix as it wasn't really P0, meh.
5 people were impacted by this yesterday after fighting had ended when a creeper blew them up, then they were vaulted.
That's why. Sure, modmail can free them, but I'd prefer to save you any future volume.
shrug The 'bug' has existed for 4 years and only now when it's strategic is the sky falling.
that's how things always work, anyways new build staged.
On Thu, Nov 12, 2015 at 11:58 AM, erocs notifications@github.com wrote:
shrug The 'bug' has existed for 4 years and only now when it's strategic is the sky falling.
— Reply to this email directly or view it on GitHub https://github.com/Civcraft/PrisonPearl/pull/75#issuecomment-156184308.
Point well made, however this is the first I personally knew of it. Woulda addressed it long ago, regardless of who reported it, had I known.
Also, we're in Ragnarok mode. Player death is way more common right now, coinciding with large-scale open warfare ... I don't want anyone else to get the shaft from this.
ACK, good to fix at any rate.
Players are getting "pearled" by non-combat deaths long after combat has ended. Easiest fix to me seems to just add respect for combat tag and remove total reliance on the damagetracker. Seems the damagetracker isn't releasing damage events appropriately; so later we can tackle root cause.
Addresses issue #74