Hard Light Productions Forums

Modding, Mission Design, and Coding => FS2 Open Coding - The Source Code Project (SCP) => Topic started by: chief1983 on January 05, 2012, 04:18:52 pm

Title: Mantis upgraded
Post by: chief1983 on January 05, 2012, 04:18:52 pm
Mantis has been upgraded to the latest stable release, 1.2.8.  No hiccups were encountered during the upgrade, but please mention if anything suddenly seems strange.
Title: Re: Mantis upgraded
Post by: Valathil on January 05, 2012, 04:20:45 pm
what if a bug prevents me from reporting that the bug reporting isnt working ? </zen>
Title: Re: Mantis upgraded
Post by: Cobra on January 05, 2012, 04:33:41 pm
And now my brain has just exploded.
Title: Re: Mantis upgraded
Post by: Valathil on January 05, 2012, 04:47:56 pm
And now my brain has just exploded.

Known issue
Title: Re: Mantis upgraded
Post by: The E on January 05, 2012, 05:13:35 pm
Will Not Fix.
Title: Re: Mantis upgraded
Post by: Cobra on January 05, 2012, 07:42:55 pm
But it's critical :(
Title: Re: Mantis upgraded
Post by: MatthTheGeek on January 06, 2012, 12:53:02 am
But not reproducible :p
Title: Re: Mantis upgraded
Post by: Cobra on January 06, 2012, 01:58:23 am
It's critical, but,
not reproducible,
It's Cobra's braaaaain!
Title: Re: Mantis upgraded
Post by: Valathil on January 06, 2012, 07:00:20 am
Can you post a log please?
Title: Re: Mantis upgraded
Post by: chief1983 on January 06, 2012, 10:01:17 am
Sorry to stop the party, but I have more actual news on this front.  The new Mantis supports a plugin for integration with various version control systems, such as SVN and Git.  I've enabled the SVN integration now, so any SVN commits that have ever referenced a ticket by number are now mentioned in the ticket's detail page under 'Related Changesets.'
Title: Re: Mantis upgraded
Post by: Valathil on January 06, 2012, 07:32:48 pm
nice work
Title: Re: Mantis upgraded
Post by: niffiwan on January 06, 2012, 09:46:20 pm
sweet  :yes:

Can the plugin also populate the [ diff ] & [ file ] links as can be seen on this page?

http://scp.indiegames.us/mantis/plugin.php?page=Source/view&id=3739
(the issue was picked at random)

edit: clarify the link names
Title: Re: Mantis upgraded
Post by: chief1983 on January 07, 2012, 01:34:11 pm
Not without WebSVN unfortunately.  We'd have to set up WebSVN on a server and have it point to our repository.  I might see how running it remotely on the SCP's website server works, since the SVN is on icculus.org and not indiegames.  I'm trying to figure out why the last commit didn't get checked in automatically first though.
Title: Re: Mantis upgraded
Post by: niffiwan on January 08, 2012, 04:11:01 am
I'm not sure if this is exactly what you're talking about, but maybe because some commits have a 4 digit mantis reference, not the full 7 digit reference?  e.g. svn 8147 (http://svn.icculus.org/fs2open?limit_changes=100&view=rev&revision=8147) & mantis 2545 (http://scp.indiegames.us/mantis/view.php?id=2545)?  Fixing this up looks like (http://stackoverflow.com/questions/304383/how-do-i-edit-a-log-message-that-i-already-committed-in-subversion) it'd require a fair bit of taylor's time (he's the SVN admin?) and thus probably isn't worth it for previous commits, but it'd be useful to remember for future commits to include the full 7 digit reference so the auto-mantis-linkage works.
Title: Re: Mantis upgraded
Post by: Echelon9 on January 08, 2012, 04:42:48 am
remember for future commits to include the full 7 digit reference so the auto-mantis-linkage works.
Do we really have to use the full 7 digit reference to each Mantis report? Seems highly unlikely coders will remember the extraneous leading zeros each time...
Title: Re: Mantis upgraded
Post by: niffiwan on January 08, 2012, 04:55:18 am
I made that statement just from the behaviour I've observed from looking through SVN and comparing relevant commits with mantis - maybe there's an option to tweak the mantis plugin?  Otherwise, yeah, I haven't bothered to include the leading zeroes in any of my commits so far either...
Title: Re: Mantis upgraded
Post by: chief1983 on January 08, 2012, 11:52:33 am
I'm pretty sure that many commits linked fine without a full 7 digit reference.  But the last commit tried to tell mantis to run the check in script, and it didn't seem to work.  So now I'm trying to figure out why.
Title: Re: Mantis upgraded
Post by: niffiwan on January 15, 2012, 06:18:33 pm
Can the plugin also populate the [ diff ] & [ file ] links as can be seen on this page?

I've noticed that these links are being populated now.  Ditto for a mantis issue being closed automatically by the relevant commit into SVN -  awesome work chief! :nod:
Title: Re: Mantis upgraded
Post by: chief1983 on January 15, 2012, 06:50:03 pm
Actually we're going to turn off auto-resolving for now, it might lead to some confusion I think.  We can re-discuss turning it on later maybe once people are more accustomed to how it's all working.  And yeah I set up WebSVN on a friend's server to get those links working.  Not the fastest but it works.
Title: Re: Mantis upgraded
Post by: Valathil on January 16, 2012, 07:23:29 am
Is this why Zacam has been stealing my resolved bugs? The auto resolver?
Title: Re: Mantis upgraded
Post by: The E on January 16, 2012, 08:02:22 am
Yeah.
Title: Re: Mantis upgraded
Post by: Valathil on January 16, 2012, 08:37:57 am
K ill call back the hitmen
Title: Re: Mantis upgraded
Post by: chief1983 on January 16, 2012, 09:42:56 am
Yeah it could be automatic, I thought I turned off auto-resolve but it still seems to be doing it.
Title: Re: Mantis upgraded
Post by: Valathil on January 23, 2012, 01:18:33 pm
Need heuristic for "dont reassign bug after status == resolved || closed" I'm losing stats fast here.
Title: Re: Mantis upgraded
Post by: chief1983 on January 23, 2012, 02:48:26 pm
FTFY.