Friday, 2009-06-19

../irclogs/#mantishelp.2009-06-19.log
--- scribe started ---00:00
mantisbotNew bug: Bug 10605 - izmirliyorumcu - open - new01:07
mantisbotNew bug: dsa - http://www.mantisbt.org/bugs/view.php?id=1060501:07
pigibi..03:54
[KK]Kirilllo all07:15
[KK]Kirillpaul_: You around?07:17
[KK]KirillWhy you change utf8_strlen to strlen? http://git.mantisforge.org/w/mantisbt.git?a=blobdiff;f=core/utility_api.php;fp=core/utility_api.php;h=037f7dc90e85a86d144ed1b3e844c4d3b50cacc8;hp=cd3f628039414b3deb6674e94e92122796c3d8b6;hb=e6ef953ccc0838eb1c43ed0acf0fb3111fa6cadb;hpb=e0051bb319a6501d4953c504af098f0a4e9859e707:18
dhx_m[KK]Kirill: because you don't really need to know the actual number of characters in the string10:04
dhx_m[KK]Kirill: you only need to know if there is a character in the string10:04
dhx_m[KK]Kirill: strlen() is perfectly safe to use with UTF8 for determining the storage size of a string... but it won't tell you the actual number of characters in the string10:04
dhx_m[KK]Kirill: UTF8 does not make use of 0x00 except to terminate strings, so strlen() can still tell you how many bytes are in a string10:05
[KK]Kirilldhx_m: ok10:05
dhx_m[KK]Kirill: all strlen() does is count how many bytes there are until a 0x00 character10:05
[KK]Kirillif first char has 0x00? What happend?10:06
dhx_m[KK]Kirill: then strlen = 0 :)10:06
[KK]Kirillok10:08
[KK]Kirilldhx_m: can you check bug 10494?10:12
mantisbotBug 10494 - Kirill - open - new10:12
mantisbotNational characters in filename of attaches - http://www.mantisbt.org/bugs/view.php?id=1049410:13
dhx_m[KK]Kirill: ok10:14
[KK]Kirilldhx_m: thanks10:14
dhx_m[KK]Kirill: will have to wait a bit though :p10:27
paul_.11:23
paul_,11:23
mantisbotNew bug: Bug 10606 - srafique - open - new11:30
mantisbotNew bug: When i do such and such quickly, so and so happens - http://www.mantisbt.org/bugs/view.php?id=1060611:30
mantisbotNew bug: Bug 10607 - cbasset - open - new12:25
mantisbotNew bug: Non-ASCII characters, collation and MS SQL Server - http://www.mantisbt.org/bugs/view.php?id=1060712:25
nuclear_eclipsemorning all12:31
dhx_mhey13:05
nuclear_eclipsehowdy13:23
* nuclear_eclipse is looking over your proposed patches for 1.2...13:24
dhx_mnuclear_eclipse: neat :)13:24
dhx_mnuclear_eclipse: I have a little bit more work to do on some of them (tomorrow)13:25
nuclear_eclipsehmm13:25
dhx_mas per the email13:25
nuclear_eclipsehow much more work?13:25
nuclear_eclipseyou expect me to *read*?!13:25
dhx_mhaha13:25
* nuclear_eclipse throws a chair13:25
dhx_mwell 10274 needs 3 minutes worth of documenting (can wait)13:26
dhx_mthat's about it really, the others should be OK I think13:26
nuclear_eclipsere 10274:13:33
nuclear_eclipseline 37713:33
nuclear_eclipseif ( config_get( 'show_scheduled_release_dates' ) ) {13:33
nuclear_eclipseif ( $t_show_version ) {13:33
nuclear_eclipseif ( config_get( 'show_scheduled_release_dates' ) ) {13:33
nuclear_eclipseseems those two lines could be merged into one conditional...13:33
dhx_mah yep13:33
dhx_mwill fix13:33
nuclear_eclipsealso, could you take some time to rebase that onto current master?13:34
dhx_myep13:34
nuclear_eclipsethanks :)13:34
dhx_mis it preferred to leave a space after statements if, foreach, etc?13:45
dhx_mseems most of the code has no space after13:45
nuclear_eclipsedhx_m: I don't think we're very consistent in that manner, but I personally *do* put a space after an if, but ironically, *not* after for/foreach/while which I see as being more like a function call... =\13:48
nuclear_eclipseit's just habit that I do in any language...13:48
dhx_mah ok13:48
dhx_mI'll just do whatever fits in with the surrounding code then :)13:49
nuclear_eclipsehowever, I think our official coding guidelines wants spaces around anything that could ever possibly have spaces around it...13:49
dhx_mok, I don't mind either way13:49
[KK]Kirilllo all13:55
[KK]KirillI have trouble13:55
[KK]KirillI don't see coments in emal13:55
[KK]Kirillcoments*13:56
[KK]Kirillemail*13:56
[KK]Kirillcomments*13:56
nuclear_eclipse[KK]Kirill: in your account preferences, what do you have set for a bugnote limit?14:02
[KK]Kirill014:03
[KK]KirillJohn, I see email14:03
[KK]Kirillbut they haven't comments14:03
[KK]Kirillbefore "BIG" commit in email I see summary issue and it's comments14:04
[KK]Kirillfor now I see only summary of issue14:04
[KK]Kirilland history of change14:04
nuclear_eclipsehmm14:05
* nuclear_eclipse will investigate..14:05
[KK]Kirillhttp://git.mantisforge.org/w/mantisbt.git?a=blobdiff;f=core/email_api.php;fp=core/email_api.php;h=4dafba5363fb0ecabbd739d39c50aa07d80b17e1;hp=a37f8f34fa91705349b3773a344e61986653425f;hb=ba1949ee7c4ec018505bb3c62a0e2e406532bc30;hpb=0bbee26ac3317246ed6c866571aa615b45e1230f14:07
nuclear_eclipsehmm, I knew that patch didn't look right....14:09
[KK]KirillI can try return back14:09
[KK]Kirilland test14:10
[KK]Kirillnot. this don't help14:14
* nuclear_eclipse breaks paul_'s knees again14:20
[KK]Kirill>:D14:21
nuclear_eclipsethis is the commit that broke it, once again broken by paul....14:21
nuclear_eclipsehttp://git.mantisforge.org/w/mantisbt.git?a=commit;h=eba305694fcf9191bad01e5d5d4065cfb04914a014:21
nuclear_eclipsepaul_: you really need to stop "adjusting" things without actually *testing* that what you've changed is still working...14:22
[KK]Kirillnuclear_eclipse: can I discard this change in my branch?14:22
nuclear_eclipsenot necessarily14:22
nuclear_eclipseI'm not even sure *why* he made that change, so I can't say whether or not another commit relies on that....14:23
[KK]Kirill:(14:23
[KK]Kirillok. this bug not serious14:24
[KK]KirillI put in bugtracker14:24
nuclear_eclipse`git revert` results in a conflict...14:24
[KK]Kirillnuclear_eclipse: Can I apply re-patch?14:25
nuclear_eclipse?14:25
[KK]KirillI can save this as patch14:26
CIA-17Mantisbt: jreese * r545da5fc106c /core/bugnote_api.php: Fix broken bugnote caching introduced in eba3056914:37
nuclear_eclipse[KK]Kirill: fixed now14:38
phl4kxhi14:47
phl4kxcan I put more than 1 email for the user in my user settings???14:47
nuclear_eclipsephl4kx: no14:51
phl4kx:(14:52
phl4kxthis is possible in the next version of mantis?14:52
nuclear_eclipseno14:53
phl4kxwhy :(14:53
phl4kx?14:53
phl4kxis not necessary?14:53
nuclear_eclipsebecause it would require a lot of changes to the way mantis works and the assumptions that it makes14:54
phl4kxah, is ok14:54
nuclear_eclipsewhy does a user need multiple email addresses?14:54
phl4kxwell14:55
CIA-17Mantisbt: hickseydr * r5824bb70420b / (26 files in 4 dirs): Remove hardcoded uses of ADMINISTRATOR constant14:55
phl4kxWe are a software consulting, and we have an ERP Software, lets our clients give us 3 email confirmation per client14:55
phl4kxlets any change with mantis, we have to confirmed to the 3 emails of our clients14:56
nuclear_eclipsewell, that still doesn't really say why a user needs multiple addresses -- that just says "system A allows us to assign multiple emails, so mantis should too"14:56
nuclear_eclipseif there's an honest reason to *need* multiple addresses, then I'm curious what the underlying "problem" / "cause" for the need, and if there's a better solution14:57
phl4kxnuclear_eclipse14:58
phl4kxwe need to assign a multiples email14:58
nuclear_eclipseI guess my point is: does a user really need to get blasted with a notification to each email address, and why not just consolidate all those addresses to a single address?14:58
phl4kxfrom different person of our client14:58
nuclear_eclipseok, that's what I'm getting at :)14:59
nuclear_eclipseyou're using a single user in mantis to represent a multi-person entity?14:59
phl4kxnot really14:59
phl4kxsingle User represent a single Client14:59
nuclear_eclipsewell, in this context, "Client" is defined as containing multiple individual persons, correct?15:00
phl4kxbut, Single Client, need a confirmation to a 3 workers..like , the logistic boss, human resource boss and the IT Manager15:00
phl4kxnuclear_eclipse...Client = Company , and company have a lot of workers15:01
phl4kxand all the Boss need a confirmation about change in the ERP Software15:02
paul_dhx_m: no15:02
phl4kxlets, we use mantis and we like to add all the email boss in mantis15:02
nuclear_eclipsephl4kx: if you create a project for each client, then add a set of users to that project, you can also configure notifiication flags to set a minimun threshold for notification, that will send notifications to everyone in the project, which would accomplish what you want using multiple user accounts....15:02
phl4kxnuclear_eclipse15:03
phl4kxwe have only 1 PROJECT15:03
phl4kxour PROJECT is the ERP SOFTWARE15:03
nuclear_eclipseplease don't shout15:03
phl4kxand in the erp software we have all the clients15:03
phl4kxwith a information level15:03
nuclear_eclipseI'm simply trying to offer other methods of achieving the end result that you want15:03
phl4kxbut only we can assign one address :(15:03
phl4kxnuclear_eclipse yes... thanks friend :)15:04
phl4kxnuclear_eclipse, lets I cant assign multiple addres for a 1 informated user?15:04
nuclear_eclipsephl4kx: if you look at configuration for $g_notify_flags, you can set a threshold for people in the projcet to receive notifications, even if they aren't assigned to the issue15:04
phl4kxah ok15:05
nuclear_eclipsethen, assuming that you set the important client persons as users meeting that threshold, they will get notified automatically15:05
paul_nuclear_eclipse: ;/15:06
nuclear_eclipseeg, if you set the notification threshold to MANAGER, and set the important client users as MANAGER on the project, then they will get notified automatically15:06
[KK]Kirillnuclear_eclipse: I see you try fix?15:06
paul_nuclear_eclipse: I did test :P15:06
paul_nuclear_eclipse: we should have a test framework :P15:06
nuclear_eclipsenot very well then, because it couldn't have possibly worked the way you had it....15:07
phl4kxnuclear_eclipse is ok15:08
phl4kxwell, definitly I cant assign multiply addres in user settings for a informated???15:08
nuclear_eclipseright15:09
nuclear_eclipsephl4kx: my only other suggestion is to set up a mailing-list or something similar that automatically broadcasts an email to one address out to multiple addresses15:10
phl4kxnuclear_eclipse, lets, im going to tell my clients to create a mai list with the 3 email boss, and I register only the mail list, and the mail server of my client reparted the emails :D15:10
phl4kxhahaha, yes the same I think :915:10
phl4kx:)15:10
phl4kxthank you!15:11
nuclear_eclipseyou're welcome15:11
[KK]Kirillnuclear_eclipse: John, thanks. It's work15:16
nuclear_eclipsenp15:16
mantisbotNew bug: Bug 10608 - nrbrown - open - new16:37
mantisbotNew bug: Display is "off" - http://www.mantisbt.org/bugs/view.php?id=1060816:37
[KK]KirillJohn, for now how sort changelog?16:39
[KK]Kirillby date or by version?16:39
[KK]Kirillsomething strange http://www.valentina-db.com/bt/changelog_page.php16:40
[KK]Kirillit's in 1.2.0 a316:49
CIA-17Mantisbt: s.mazeland * r5558ad916175 /plugins/MantisGraph/lang/strings_english.txt: Fix typo in message (time->times)16:56
[KK]Kirillsiebrand: lo16:56
siebrandhola16:56
[KK]KirillHow are you?16:57
siebrandheh, struck user names.16:57
siebrandshouldn't be too hard to track down.16:57
[KK]Kirill:)16:57
siebrand[KK]Kirill: that's git master?16:57
nuclear_eclipse[KK]Kirill: changelog is sorted by (status,last-updated), and changelog is just sorted by (last-updated)16:57
[KK]Kirillnuclear_eclipse: this not changed from 1.2.0 a3?16:58
nuclear_eclipsenope16:59
[KK]Kirillnuclear_eclipse: can you see link http://www.valentina-db.com/bt/changelog_page.php16:59
nuclear_eclipseyeah, what's wrang about that?16:59
[KK]KirillI see status16:59
[KK]KirillI see last update16:59
[KK]Kirillbut version jumping17:00
[KK]Kirillkernel 2.0 b25, after kernel 2.0 b35 than 2.0.b2717:00
nuclear_eclipseI see b25, then b26, then b27....17:01
[KK]Kirilljust see 1 issue from each version: status and last update17:01
nuclear_eclipseit all looks correctly ordered to me...17:01
[KK]Kirillbrrrr17:01
[KK]Kirillone sec17:01
nuclear_eclipsehttp://imgur.com/2J4mS.png17:02
nuclear_eclipsehttp://imgur.com/2J4mS.png17:03
[KK]Kirillsorry. miranda-crash17:03
[KK]Kirillhttp://itrash.ru/idb/174/ovolga.png17:03
[KK]Kirillwhy?17:04
[KK]KirillI return after 5 minutes17:04
nuclear_eclipseno clue, it shows up correctly for me... http://imgur.com/8WAcC.png17:05
[KK]Kirillnuclear_eclipse: strange. in first screen before b25 has version, but on second - not17:18
[KK]KirillI select project kernel and all version show normaly17:19
[KK]Kirillbut when I return to All project I see such as I send screenshot17:21
nuclear_eclipsehmm, I see that17:22
nuclear_eclipseI'm sure I can knock paulr over the head on this one again...17:22
nuclear_eclipsehe's the one that broke roadmap sort orders recently17:23
[KK]Kirillok17:23
nuclear_eclipse[KK]Kirill: mind modifying your install a bit to help me debug your problom?18:14
[KK]Kirillvalentine-db?18:14
nuclear_eclipseyeah18:14
[KK]Kirillsorry, I haven't right to this site18:14
nuclear_eclipseah, ok18:14
[KK]Kirillit's russian peolpe ask me18:14
nuclear_eclipseis it running latest git master?18:15
[KK]Kirilland I retranslate problem18:15
[KK]Kirillnot18:15
[KK]Kirill1.2.0 a318:15
nuclear_eclipseah, ok; that's the problem I think18:15
[KK]KirillI wrote him, that he must test on last git18:15
nuclear_eclipseversion caching wasn't using the proper sort order for a long time, and I fixed that somewhat recently (couple weeks ago iirc)18:16
[KK]Kirillhe test this and I write what happed18:17
[KK]Kirillhappened18:17
CIA-17Mantisbt: s.mazeland * rbc290c3ebe1a / (20 files in 4 dirs): Localisation updates from http://translatewiki.net (2009-06-19 19:35 UTC)19:50
nuclear_eclipsesiebrand: you about?19:51
siebrandnuclear_eclipse: obviously.19:53
nuclear_eclipse:)19:55
nuclear_eclipsesiebrand: can you inform me of the impact/feasibility of bug 10598 on translations?19:57
mantisbotBug 10598 - dhx - open - assigned19:57
mantisbotShorten HTML button strings shown when viewing/updating a bug - http://www.mantisbt.org/bugs/view.php?id=1059819:57
siebrandhmm19:57
nuclear_eclipsehttp://git.mantisforge.org/w/mantisbt/dhx.git?a=commitdiff;h=00023beabcaacb71091871ec5d7fba45632af1a919:57
nuclear_eclipsethat's the proposed commit for the issue, and I'm in favor of the move because I'm all for reducing redundant text, and for making the UI smaller/cleaner19:58
nuclear_eclipsebut I'm just wanting to know how big of an impact that change would have on translations19:58
siebrandnuclear_eclipse: I guess I have to agree.19:59
siebrandnuclear_eclipse: that will possibly not work for every language, but that's a problem for that language.19:59
siebrandnuclear_eclipse: Would it also be possible that one button is split over multiple lines?19:59
nuclear_eclipseI mean obviously, changing the english text has no immediate impact on the translations, but I'm curious how much effort would be involved to make the translations consistent with the changes to the english text19:59
siebrandnuclear_eclipse: in that case, we could consider processing button text, and replacing whitespace by no breaking spaces.19:59
siebrandnon breaking spaces, eveb20:00
siebrandnuclear_eclipse: we 'fuzzy' the translations, and translators will eventually find the tagged messages, and do something with it.20:00
siebrandnuclear_eclipse: preferably make certain that you make a complete change set, as I expect some 10-15 localisations to be updated quite soon.20:01
nuclear_eclipseassuming that we go ahead with the change for a 1.2rc release tomorrow, would it be feasible to have translators get follow-up changes in by an eventual final release?20:02
siebrandnuclear_eclipse: yes, of course.20:02
nuclear_eclipseok20:02
nuclear_eclipse< siebrand> nuclear_eclipse: preferably make certain that you make a complete change set20:03
siebrandnuclear_eclipse: I do need a 2 day or so heads up to be able to export and commit translations, and preferably a two weeks heads up so I can make some fuss about the upcoming MantisBT release, and asking translators to complete their translations.20:03
nuclear_eclipseI'm not grokking that20:03
siebrandnuclear_eclipse: well, just click a bit through the UI to check if there is more redundancy.20:04
nuclear_eclipseah20:04
* siebrand grumbles20:05
nuclear_eclipseuh oh20:05
siebrandhow do I tell git to add all files that are new in my tree?20:05
siebrandgit add . does not do the trick.20:05
nuclear_eclipseo rly?20:05
nuclear_eclipseare you sure you're in the base of the repo when you do that?20:06
siebrandyep20:06
siebrand(on windows, though)20:06
nuclear_eclipsehmm20:06
nuclear_eclipseI've honestly never used git on windows =\20:07
siebrandwell, it works with :20:07
siebrandC:\branches\mantisbt>git add plugins\XmlImportExport\lang20:07
siebrandbut I had to do it for all 3 plugins.20:07
siebrandThis sucks.20:07
nuclear_eclipseI wonder if windows CLI doesn't treat '.' appropriately20:08
* nuclear_eclipse suggests `git add plugins` ?20:08
CIA-17Mantisbt: s.mazeland * ra51034bd95b8 /plugins/ (21 files in 3 dirs): Localisation updates from http://translatewiki.net (2009-06-19 19:35 UTC)20:09
siebrandnuclear_eclipse: that's not a bad idea. Will try that next time.20:09
siebrandAnyway, our localisations for plugins just tripled :)20:09
nuclear_eclipseyay20:10
nuclear_eclipsenow, for something completely different20:10
nuclear_eclipsewhat is the outlook for translated manuals?20:10
siebrand(afk)20:10
siebrandNothing from my side.20:10
siebrandNo docbook support.20:10
nuclear_eclipseok20:10
nuclear_eclipseso any translations done would have to be all done by-hand basically?20:11
siebrandif someone writes docbook file format support for the MediaWiki translate extension, that would be sweet :)20:11
siebrandyes20:11
nuclear_eclipseok20:11
* nuclear_eclipse is not writing that :*20:12
nuclear_eclipse:P20:12
siebrandnuclear_eclipse: there allegedly is a docbook to gettext convertor.20:14
siebrandnuclear_eclipse: can handle gettext.20:15
nuclear_eclipseoh dear, that sounds like it would be more trouble than by-hand translations... :P20:15
siebrandnuclear_eclipse: http://po4a.alioth.debian.org/ looks like it may be able to do it.21:25
paul_moo22:31
paul_nuclear_eclipse: i think i've concluded what I think about utf8 :022:31

Generated by irclog2html.py