MoinMoin: MoinMoinChat/Logs/moin-dev/2013-07-17

   1 2013-07-17T02:41:44  <asingla> ThomasWaldmann: updated + replied to your comments https://codereview.appspot.com/10915044/
   2 2013-07-17T03:51:07  <ananasova> moin
   3 2013-07-17T03:54:33  <ananasova> ThomasWaldmann: so the rule holds for lists. i am  it also to tuples
   4 2013-07-17T03:54:57  <ananasova> i am assuming it should be used also for tuples
   5 2013-07-17T03:57:12  <ananasova> ThomasWaldmann: about "incompatibility issues" - https://tldrlegal.uservoice.com/forums/179175-general/suggestions/3760887-licensense-compatibility
   6 2013-07-17T03:58:31  * ananasova has upvoted
   7 2013-07-17T05:09:06  * xiaq_ added a diary entry
   8 2013-07-17T05:09:08  *** xiaq_ is now known as xiaq
   9 2013-07-17T06:53:00  <sharky93> moin
  10 2013-07-17T06:53:07  <sharky93> some updates @ https://github.com/twitter/bootstrap/issues/2054
  11 2013-07-17T06:55:20  <sharky93> and all they say is sooooon @twbootstrap
  12 2013-07-17T07:02:25  <ananasova> "There's pretty much no chance the bootstrap license will ever change."
  13 2013-07-17T07:03:32  <ananasova> ohh sorry, the issue is so looong... didn't see initially the issue reopened
  14 2013-07-17T07:05:27  * sharky93 enjoys the rain :)
  15 2013-07-17T07:09:02  *** vpv_ has joined #moin-dev
  16 2013-07-17T07:14:09  *** vpv has quit IRC
  17 2013-07-17T07:29:55  <rciorba> moin
  18 2013-07-17T07:38:11  <rciorba> please look at the latest: https://codereview.appspot.com/11254043/ ok for a PR?
  19 2013-07-17T07:42:27  <xiaq> ThomasWaldmann: separate +show view for tickets https://codereview.appspot.com/11430043/ also see my message there.
  20 2013-07-17T07:42:30  <xiaq> eSyr: ^^
  21 2013-07-17T07:42:33  <xiaq> eSyr-ng: ^^
  22 2013-07-17T09:37:19  *** greg_ has joined #moin-dev
  23 2013-07-17T09:52:39  <sharky93> TheSheep: so should we fix the indent all in one changeset?
  24 2013-07-17T09:53:13  <sharky93> TheSheep: it might be better, if you could confirm the previous commits :)
  25 2013-07-17T09:58:11  <TheSheep> sharky93: "confirm"?
  26 2013-07-17T09:59:13  <sharky93> TheSheep: review i mean.
  27 2013-07-17T10:29:23  <sharky93> TheSheep: what is the norm to be followed for bringing in the changes you suggested ? more commits ? if yes, what message should be used?
  28 2013-07-17T10:51:52  <TheSheep> sharky93: I don't know
  29 2013-07-17T10:56:44  <sharky93> TheSheep: well :D .. maybe we ask thomas
  30 2013-07-17T11:00:19  <sharky93> ThomasWaldmann: ^^
  31 2013-07-17T11:27:42  <sharky93> TheSheep: btw, rest of the commits are fine ?
  32 2013-07-17T12:23:40  <TheSheep> sharky93: no, I'm still looking
  33 2013-07-17T12:23:47  <TheSheep> sharky93: sorry, lots of work a work
  34 2013-07-17T12:28:55  <ananasova> ThomasWaldmann, waldi: updated Subscribers cr, please review - https://codereview.appspot.com/10950045/
  35 2013-07-17T12:30:13  <ThomasWaldmann> sharky93: your question is unclear
  36 2013-07-17T12:47:07  *** dave_largo has joined #moin-dev
  37 2013-07-17T12:58:02  *** brunomartin has joined #moin-dev
  38 2013-07-17T13:11:56  <TheSheep> sharky93: reviewed the rest
  39 2013-07-17T13:28:26  <ananasova> ThomasWaldmann: updated https://codereview.appspot.com/10950045/
  40 2013-07-17T13:48:14  *** rob_____ has joined #moin-dev
  41 2013-07-17T13:49:47  <rob_____> hello everyone, could someone tell me when moin2 will -approximately- be released? will it rather be in 4 months or in 1+ years? i'm considering to use it for a project which will take a few months anyway, however if moin2 will be released later than that, i'd rather use 1.9.7
  42 2013-07-17T13:53:00  * ThomasWaldmann can't predict the future
  43 2013-07-17T13:53:47  <ThomasWaldmann> the general rule is that if you need something for production, you should use 1.9.x
  44 2013-07-17T13:54:39  <rob_____> couldn't you make an educated guess, though?
  45 2013-07-17T13:55:16  <rob_____> I suppose your judgment on how long stuff on the to-do list will take is worth more than mine :)
  46 2013-07-17T13:58:19  <ThomasWaldmann> it depend on contributions and i can't predict those.
  47 2013-07-17T13:58:48  <rob_____> I see. thanks
  48 2013-07-17T13:59:00  <ThomasWaldmann> but considering that soc will run for a while and we will be busy merging the stuff afterwards, don't expect it too soon.
  49 2013-07-17T13:59:24  <rob_____> alright, thank you
  50 2013-07-17T13:59:26  <rob_____> bye
  51 2013-07-17T13:59:46  *** rob_____ has quit IRC
  52 2013-07-17T14:28:56  <xiaq> ThomasWaldmann: comments on this too, plz :) https://codereview.appspot.com/10889044/
  53 2013-07-17T14:28:59  <xiaq> eSyr-ng: ^^
  54 2013-07-17T14:40:54  <ThomasWaldmann> xiaq: the generalized index item type sounds interesting
  55 2013-07-17T14:42:15  <ThomasWaldmann> and that you work on getting some stuff published on your public repo
  56 2013-07-17T14:56:51  <xiaq> ThomasWaldmann: keeping patchsets in the mq makes for a very clean history :)
  57 2013-07-17T15:00:27  <ThomasWaldmann> midterm evals will be based on committed and published stuff, not on whatever students have in codereview or mq
  58 2013-07-17T15:02:23  * xiaq 'll try to publish more then
  59 2013-07-17T15:07:08  <ThomasWaldmann> also keep in mind that mentors won't actually run your code as long as they can't get it easily (like when cloning your public repo)
  60 2013-07-17T15:07:38  * ThomasWaldmann bbl
  61 2013-07-17T15:25:07  <sharky93> ThomasWaldmann: for ex here @ https://bitbucket.org/sharky93/moin-2.0-bootstrap/commits/5d61b26cad7c7ed173053de53967f96ecb1f4372 is there a norm i should follow to include the changes suggested by TheSheep like referring to this commit in the new one or some standard message we use ?
  62 2013-07-17T15:56:33  <ananasova> ThomasWaldmann: https://codereview.appspot.com/10950045/ made some fixes, last 3 patches
  63 2013-07-17T16:24:46  <ThomasWaldmann> sharky93: i still have no idea what the problem is
  64 2013-07-17T16:25:57  <ThomasWaldmann> TheSheep: i wanted something visible to indicate this error condition, thus not just ''
  65 2013-07-17T16:30:18  <ThomasWaldmann> sharky93: something like "... for changeset XXXXXXXXX" (XXXXX being the hex id)?
  66 2013-07-17T16:31:12  <sharky93> ThomasWaldmann: yeah, ok. it was not much of a problem but just a suggestion i wanted, since obviously the history will not be very clean :)
  67 2013-07-17T16:34:23  *** ebo^ has joined #moin-dev
  68 2013-07-17T16:41:52  <sharky93> TheSheep: did you try cloning and running an instance of the bootstrap fork ?
  69 2013-07-17T16:47:50  <TheSheep> sharky93: yes, but not today
  70 2013-07-17T16:48:48  *** rciorba has quit IRC
  71 2013-07-17T16:50:11  <sharky93> TheSheep: hmm, i was wondering what should we go for next
  72 2013-07-17T17:42:35  <asingla> ThomasWaldmann: updated https://codereview.appspot.com/10915044/
  73 2013-07-17T17:43:08  *** rciorba has joined #moin-dev
  74 2013-07-17T17:47:15  *** greg_ has quit IRC
  75 2013-07-17T17:52:46  <asingla> ThomasWaldmann: also updated class name https://codereview.appspot.com/10869045
  76 2013-07-17T18:16:12  <sharky93> TheSheep: indents and fixing pep8 etc is one, but after that
  77 2013-07-17T18:16:15  <sharky93> ?
  78 2013-07-17T18:16:23  <sharky93> TheSheep: is it early to move onto specific views ?
  79 2013-07-17T21:13:11  *** dave_largo has quit IRC
  80 2013-07-17T21:31:36  *** ebo^ has quit IRC
  81 2013-07-17T21:42:13  *** rciorba has quit IRC
  82 2013-07-17T22:00:11  *** brunomartin has quit IRC
  83 2013-07-17T22:20:49  *** bilal has joined #moin-dev
  84 2013-07-17T22:33:48  *** brunomartin has joined #moin-dev
  85 2013-07-17T22:42:35  *** brunomartin has quit IRC
  86 2013-07-17T22:47:31  *** bilal has quit IRC
  87 2013-07-17T23:19:12  * asingla works on how to handle the root url in different scenarios.
  88