| ../irclogs/#mantishelp.2009-06-06.log | ||
| --- scribe started --- | 00:00 | |
| paul_ | hmm | 00:02 |
|---|---|---|
| paul_ | apc? | 00:02 |
| nuclear_eclipse | go back to sleep | 00:02 |
| nuclear_eclipse | jreese@mach[~/workspace/mantisbt] git bisect good < 20:02 | 00:03 |
| nuclear_eclipse | 1096c049774b5f6e85813c8f7d7bb37eea33dd1f is first bad commit | 00:03 |
| nuclear_eclipse | commit 1096c049774b5f6e85813c8f7d7bb37eea33dd1f | 00:03 |
| nuclear_eclipse | Author: Siebrand Mazeland <s.mazeland@xs4all.nl> | 00:03 |
| nuclear_eclipse | Date: Sun May 3 17:57:23 2009 +0200 | 00:03 |
| nuclear_eclipse | * remove some unneeded strtolower() | 00:03 |
| nuclear_eclipse | :100644 100644 823f8f347e0ec7fae84038cffda5eda6b69fb88d 636c439d9e79b35c68cfa9872389963f5077fb91 Mchangelog_page.php | 00:03 |
| nuclear_eclipse | :100644 100644 21b405f1b3fc11d20203076bf3bac3ba6f9874f8 e599aedf198bfc8ed28c7e806aad9adc0abe123e Mroadmap_page.php | 00:03 |
| nuclear_eclipse | it's quite obvious now :P | 00:04 |
| nuclear_eclipse | or rather, it's obvious that you broke something | 00:04 |
| nuclear_eclipse | I still haven't figured out what the actual bug is.... | 00:04 |
| siebrand | now I'm more confused. | 00:05 |
| nuclear_eclipse | lol | 00:05 |
| siebrand | We have established that the 'live' version is broken, but master is not. | 00:05 |
| siebrand | Is the right? | 00:05 |
| nuclear_eclipse | right, that's the commit that "fixed" it :) | 00:05 |
| siebrand | OK. I do not see what's wrong with that commit. Please teach me, oh master :) | 00:06 |
| nuclear_eclipse | in this context, `bad` means it exhibits the current good behavior (ie, I'm working in the context of git-bisect) | 00:06 |
| nuclear_eclipse | got it now | 00:06 |
| siebrand | (except that I forgot to note the adding of the space in the commit msg) | 00:06 |
| nuclear_eclipse | there *was* a bug, and you adding the word_separator actually fixed it ;) | 00:06 |
| siebrand | ah, less confused. | 00:06 |
| siebrand | Conclusion: much ado 'bout nuttin. | 00:06 |
| nuclear_eclipse | or rather, you apparently fixed it by accident :P | 00:07 |
| siebrand | nope, I didn't. That was done on purpose, because the lack of space annoyed me. | 00:07 |
| siebrand | I just forgot to note it on the commit. | 00:07 |
| nuclear_eclipse | the confusion is that the live install is officially stale :D | 00:07 |
| siebrand | *past* bed time for me. | 00:07 |
| siebrand | Cheers! | 00:07 |
| nuclear_eclipse | cheers siebrand | 00:07 |
| CIA-61 | Mantisbt: s.mazeland live * rc53b7f80f958 / (changelog_page.php roadmap_page.php): * remove some unneeded strtolower() | 00:09 |
| nuclear_eclipse | quit | 00:11 |
| paul_ | hmm | 00:14 |
| paul_ | why are we patching live? | 00:14 |
| paul_ | in fact, why does live exist? ;/ | 00:15 |
| nuclear_eclipse | because you broke dates and still havent fixed them... | 00:18 |
| paul_ | hmm | 00:26 |
| paul_ | I made a change a while back | 00:26 |
| paul_ | to let you change globalconfig settings within pages | 00:26 |
| paul_ | that's not a great idea with plugins now available or is it? | 00:26 |
| nuclear_eclipse | what would be the point of it? | 00:27 |
| paul_ | http://git.mantisforge.org/w/mantisbt.git?a=commitdiff;h=d26d7cf6e8d7050cf7eba4793fcc0296a93a8e95 | 00:27 |
| nuclear_eclipse | cant check that atm... =/ | 00:28 |
| * paul_ tries to recall | 00:33 | |
| paul_ | where/when do we auth a user | 00:33 |
| paul_ | public function __get( $t_string ) { | 00:37 |
| paul_ | if( is_null( $this->{$t_string} ) ) { | 00:37 |
| paul_ | $this->{$t_string} = config_get( $g_default_mapping[$t_string], null, $this->pref_user_id, $this->pref_project_id ); | 00:37 |
| paul_ | should that also check for empty? ;/ | 00:37 |
| paul_ | i.e. "" | 00:37 |
| paul_ | ahh right | 00:50 |
| paul_ | we authenticate people for the first time by calling collapse_cache_token! | 00:51 |
| paul_ | nn | 01:00 |
| * paul_ sighs loudly | 11:12 | |
| paul_ | . | 17:35 |
| nuclear_eclipse | hi paul_ | 18:11 |
| setuid | Question: I'm trying to figure out where in the Mantis config, I'd supply our proxy configuration. We run Squid behind everything here to accelerate Apache and other apps. Where would I plug that into Mantis, so it can cache its pages in Squid as well? | 18:50 |
| nuclear_eclipse | setuid: you generally wouldn't want your mantis pages to be cached by squid | 18:51 |
| setuid | In other words, a request for a page from Mantis, goes to Squid before MySQL | 18:51 |
| setuid | nuclear_eclipse, We most-definitely do | 18:51 |
| setuid | nuclear_eclipse, Squid is configured appropriately, and there's no reason why they can't be cached. Our benchmarks show a 700% performance improvement doing so with other php-base apps. | 18:51 |
| nuclear_eclipse | that would be serving potentially stale informaton then, esp after updating an issue | 18:52 |
| setuid | Server disconnected me... | 18:53 |
| setuid | I missed anything you might have said after I said: | 18:53 |
| setuid | nuclear_eclipse, Squid is configured appropriately, and there's no reason why they can't be cached. Our benchmarks show a 700% performance improvement doing so with other php-base apps. | 18:53 |
| nuclear_eclipse | that would be serving potentially stale informaton then, esp after updating an issue | 18:54 |
| setuid | Nope, never serves stale information. | 18:54 |
| setuid | That's precisely the point | 18:54 |
| setuid | Squid knows when a page has changed, and serves up the appropriate page as needed.. either from cache, or from MySQL (and caches that) | 18:55 |
| nuclear_eclipse | then let's just say Mantis doesn't yet play nice with caching headers | 18:55 |
| nuclear_eclipse | if you really are after performance, you could try setting up an opcode cache for PHP, which will give you comparable performance gains | 18:56 |
| setuid | Wow, Freenode is flaky this morning | 18:56 |
| kenguest | morning? where? | 18:56 |
| setuid | We don't have a large system... maybe 3,000 issues filed, 1,000 or so active users. | 18:57 |
| nuclear_eclipse | 14:55 < nuclear_eclipse> then let's just say Mantis doesn't yet play nice with caching headers | 18:58 |
| nuclear_eclipse | 14:56 < nuclear_eclipse> if you really are after performance, you could try setting up an opcode cache for PHP, which | 18:58 |
| nuclear_eclipse | will give you comparable performance gains | 18:59 |
| paul_ | setuid: you run apc? | 19:00 |
| setuid | paul_, yes | 19:00 |
| paul_ | nuclear_eclipse: I hate mantis | 19:00 |
| nuclear_eclipse | good for you paul_ | 19:00 |
| paul_ | remind me where you run mantis again | 19:00 |
| setuid | moi? | 19:00 |
| paul_ | ya | 19:00 |
| * paul_ assumes it's something public | 19:00 | |
| setuid | bugs.plkr.org, bugs.pilot-link.org, bugs.jpilot.org | 19:00 |
| paul_ | nuclear_eclipse: so, question for you | 19:00 |
| paul_ | in the lifetime of a request | 19:01 |
| paul_ | within what function do you think authentication first happens? | 19:01 |
| paul_ | or gets 'checked' | 19:01 |
| nuclear_eclipse | 20:50 < paul_> ahh right | 19:01 |
| nuclear_eclipse | 20:51 < paul_> we authenticate people for the first time by calling collapse_cache_token! | 19:01 |
| nuclear_eclipse | it's only incidental though | 19:02 |
| nuclear_eclipse | ie, it just happens to be the first portion of the code that calls an API that through some chain looks for authentication info | 19:02 |
| Heady| | is there already a page which lists all available 1.2 plugins? | 19:04 |
| nuclear_eclipse | git.mantisforge.org is the closest to that | 19:05 |
| paul_ | anyway i've fixed versions | 19:09 |
| nuclear_eclipse | took you long enough :P | 19:10 |
| paul_ | but we've got some things to fix | 19:11 |
| paul_ | nusoap needs fixing | 19:11 |
| * paul_ can't test soap stuff as nusoap fails | 19:12 | |
| setuid | paul_, Any comments on performance on our side? | 19:12 |
| paul_ | setuid: i'm surprised you'd need to use squid to cache the mantis pages | 19:13 |
| paul_ | if your running apc | 19:13 |
| paul_ | although, might be worth while seeing what 1.2 does in a few weeks | 19:13 |
| paul_ | or months | 19:13 |
| setuid | We don't "need" it, but it gives us a significant performance boost | 19:13 |
| paul_ | brbfood | 19:14 |
| setuid | Is there any way to allow summary_graph_byseverity_mix.php and friends to be accessible by Guest? | 19:15 |
| setuid | i.e. not require a login? | 19:15 |
| nuclear_eclipse | setuid search config_defaults for info about designating an account for anonymous access | 19:15 |
| setuid | We have one, for browsing around the tracker | 19:15 |
| setuid | But apparently the graphs require login | 19:16 |
| nuclear_eclipse | ? that's odd | 19:16 |
| nuclear_eclipse | perhaps you could try looking in configs for thresholds to lower to VIEWER | 19:16 |
| setuid | Yep, summary isn't available to anonymous users | 19:16 |
| setuid | ah! g_view_summary_threshold | 19:17 |
| setuid | perfect... | 19:17 |
| giallu | paul_, what kind of nusoap fix? please DON'T touch upstrema code... | 19:54 |
| nuclear_eclipse | +1 | 19:55 |
| * giallu has just been asked to move nusoap out of mantis package in fedora... | 19:56 | |
| paul_ | why? | 19:59 |
| nuclear_eclipse | because NuSOAP != MantisBT | 20:01 |
| giallu | becasue if nusoap has bug, they are fixing it. just report it, better with a patch | 20:01 |
| paul_ | i meant recording moving it out of package | 20:02 |
| paul_ | regarding even | 20:02 |
| giallu | paul_, I tried to explain it to you sever times. I failed | 20:02 |
| giallu | several | 20:02 |
| giallu | suffice to say, we have another package with the same embedded lib, we want to avoid duplication | 20:04 |
| paul_ | anyway, we need to get a fixed version of nusoap | 20:12 |
| giallu | paul_, ok. I didn't catch what's the problem with nusoap | 20:19 |
| paul_ | generates php strict warnings | 20:19 |
| giallu | ok | 20:20 |
| paul_ | although i'm not quite sure where nusoap is being developed now | 20:21 |
| giallu | http://code.google.com/p/nusoap-for-php5/ here maybe? | 20:21 |
| paul_ | nope | 20:21 |
| paul_ | that's a fork | 20:21 |
| paul_ | http://sourceforge.net/projects/nusoap/ | 20:21 |
| paul_ | but also | 20:21 |
| paul_ | there's a 1.0rc version of nusoap at http://projects.ez.no/nusoap | 20:22 |
| giallu | how come we use the crappiest libraries around.... | 20:22 |
| giallu | :P | 20:22 |
| giallu | you first link doesn't work here | 20:23 |
| giallu | :( | 20:23 |
| paul_ | sf.net is down i think | 20:23 |
| giallu | all of it? | 20:23 |
| giallu | cool | 20:23 |
| * paul_ shrugs | 20:23 | |
| nuclear_eclipse | tell me again why we aren't using PHP's built-in SOAP libraries? | 20:30 |
| giallu | nuclear_eclipse, good question | 20:33 |
| giallu | I think it has something to do with the fact it's not a default component of php | 20:34 |
| nuclear_eclipse | so? :P | 20:34 |
| giallu | for me it's irrelevant | 20:34 |
| nuclear_eclipse | if someone really wants to use SOAP with Mantis, why don't we just require them to have the extension enabled? | 20:34 |
| giallu | I have it with a "yum install php-soap" | 20:35 |
| nuclear_eclipse | exactly | 20:35 |
| giallu | now | 20:35 |
| giallu | I wonder why for instance even ZF is providinig a Zend_Soap class... | 20:36 |
| nuclear_eclipse | because Zend decided to provide everything | 20:36 |
| nuclear_eclipse | and to make it all as verbose as possible while they were at it | 20:37 |
| giallu | well, ok, but if they felt compelled to wrap php-soap I bet there is a better reason | 20:37 |
| giallu | nuclear_eclipse, no, verbosity champion is Microsoft | 20:37 |
| nuclear_eclipse | perhaps, C# + .Net is les verbose than Java + Java Lib | 20:38 |
| nuclear_eclipse | C#/.Net is probably my least hated thing to ever come out of Microsoft, with a possible exception for the gaming studios they published for... | 20:39 |
| nuclear_eclipse | actually, C# combined with Xna actually makes for a really nice framework for making video games; it's just a total shame that it only ever works on Windows and Xbox... | 20:40 |
| paul_ | nuclear_eclipse: apparently the php stuff doesn't do some thing | 21:06 |
| CIA-61 | Mantisbt: paul * re17bd57d7534 /graphs/ (8 files): Remove unused graph files (half of these don't work) | 22:04 |
| CIA-61 | Mantisbt: paul * r5343a7f722a5 / (4 files in 3 dirs): Store User id of users adding files (work towards #0003437) | 22:04 |
| nuclear_eclipse | paul_: just fyi, you can't just put the bug # in, you need to do "fix|issue|bug #" type of thing | 22:06 |
| nuclear_eclipse | or at least, if you want the source integration to pick it up as a bug reference :P | 22:07 |
| paul_ | did was use \ to escape % in config api? | 22:09 |
| nuclear_eclipse | ye | 22:10 |
| paul_ | oops | 22:10 |
| paul_ | grr | 22:10 |
| paul_ | that was wrong ;/ | 22:11 |
| paul_ | I can't amend commit i've pushed can I? ;/ | 22:11 |
| giallu | bug 3437 | 22:11 |
| mantisbot | Bug 3437 - mjacek - open - acknowledged | 22:11 |
| mantisbot | User should be able to edit his own documentation - http://www.mantisbt.org/bugs/view.php?id=3437 | 22:11 |
| nuclear_eclipse | not without cauing headaches to anyone who's pulled from it, including mantsiforge | 22:11 |
| nuclear_eclipse | don't worry bout it | 22:12 |
| giallu | paul_, IMHO commit 648f388 is wrong | 22:17 |
| paul_ | oh? | 22:18 |
| nuclear_eclipse | what's commit 648f388? | 22:19 |
| giallu | One goal in the newlayout world is to get rid of g_core_path | 22:19 |
| giallu | http://git.mantisbt.org/?p=mantisbt.git;a=commit;h=648f388e1c076c431495aa2b18f77d9d5adb6ba6 | 22:19 |
| nuclear_eclipse | oh, an old news commit :P | 22:19 |
| giallu | yeah, I just noticed it | 22:19 |
| giallu | because it conflicts with my current stuff :( | 22:20 |
| paul_ | you need to allow users to move core path outside of webroot | 22:20 |
| paul_ | iirc, that's what that was fixing | 22:21 |
| giallu | when I'm done, core will be outside webroot... | 22:21 |
| giallu | but it's ok... I'll deal with that later | 22:22 |
| paul_ | well, it should be optional | 22:22 |
| paul_ | personally, I use mantis with core in webroot | 22:22 |
| paul_ | as it's easier to set up | 22:22 |
| paul_ | or more, I dont mind as long as one can just extract mantis and run it | 22:24 |
| paul_ | without requiring server config access | 22:24 |
| CIA-61 | Mantisbt: paul * r544e7d9a8cd1 /core.php: add autoload function to spl register - this allows plugins to define their own (additional) autoload functionality | 22:39 |
| CIA-61 | Mantisbt: paul * r8663867b27e3 / (9 files in 3 dirs): Changes for bug 10565 | 22:39 |
| mantisbot | Bug 10565 - jreese - open - assigned | 22:39 |
| mantisbot | User input date strings are munged with new schema - http://www.mantisbt.org/bugs/view.php?id=10565 | 22:40 |
| paul_ | nuclear_eclipse: can we fix tests? | 22:41 |
| siebrand | paul_: inconsistent message formatting in your latest commit: | 23:01 |
| siebrand | $MANTIS_ERROR[ERROR_INVALID_SORT_FIELD] = 'Invalid sort field.'; | 23:01 |
| siebrand | +$MANTIS_ERROR[ERROR_INVALID_DATE_FORMAT] = 'Invalid Date Format.'; | 23:01 |
| paul_ | ohhi | 23:03 |
Generated by irclog2html.py