Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Oct 07 16:02
    DarwinAwardWinner opened #174
  • Jun 15 12:12
    DarwinAwardWinner opened #173
  • May 26 15:49
    DarwinAwardWinner closed #171
  • May 26 15:49

    DarwinAwardWinner on master

    Enable tests in emacs-snapshot … Eliminate explicit version chec… Fix recursive autoload issue and 3 more (compare)

  • May 20 15:35

    DarwinAwardWinner on bleeding-edge

    Fix a typo in a docstring (compare)

  • May 20 14:35
    DarwinAwardWinner closed #172
  • May 20 14:35
    DarwinAwardWinner commented #172
  • May 19 02:47
    wyuenho commented #172
  • May 18 15:15
    DarwinAwardWinner commented #172
  • May 18 13:06
    wyuenho commented #172
  • May 16 23:48
    DarwinAwardWinner commented #172
  • May 16 23:18
    DarwinAwardWinner commented #172
  • May 16 21:29
    wyuenho opened #172
  • Mar 11 18:38
    DarwinAwardWinner closed #165
  • Mar 11 18:38
    DarwinAwardWinner commented #165
  • Mar 11 18:36
    DarwinAwardWinner commented #165
  • Mar 11 13:42

    DarwinAwardWinner on bleeding-edge

    Switch to MELPA unstable for te… Improve code for detecting bug … (compare)

  • Mar 11 01:30
    DarwinAwardWinner commented #171
  • Mar 11 01:30

    DarwinAwardWinner on bleeding-edge

    Eliminate explicit version chec… Fix recursive autoload issue (compare)

  • Mar 11 01:15

    DarwinAwardWinner on bleeding-edge

    Fix recursive autoload issue (compare)

Ryan C. Thompson
@DarwinAwardWinner
Which I managed to do several times while testing upgrading ido-ubiquitous to this new version
Ditto for call-interactively and called-interactively-p, which also tend to break everything if you break them
Jonas Bernoulli
@tarsius
It's always fun having to kill Emacs from the outside :-)
I am just wondering if you couldn't save a lot of work if you just patched ido.el instead.
Ryan C. Thompson
@DarwinAwardWinner
Somehow I got Emacs into a state where even pressing C-g oiverflowed the stack
Oh wow, magit next is prettier
I should upgrade :)
Jonas Bernoulli
@tarsius
Yup
What theme are you using?
Ryan C. Thompson
@DarwinAwardWinner
It’s a default Emacs with just magit and ido-ubiquitous loaded
On OSX
White BG, black text
Jonas Bernoulli
@tarsius
In case you usually use zenburn or solarized-light, then you should try my forks of these themes. These changes will be merged eventually, but for now you have to get the magit-next branches from my fork. Solarized-dark should also look okay, but I haven't actually tried making it look good yet. The default themes both in the dark and light variants I have already themed too (that's what you are currently using), but that has fallen behind a bit by now.
Anyway how should we proceed? Looks like I will have to depend on ido-ubiquitious for ido support, not just ido-completing-read+. But I won't actually add that as a dependency of the package, to avoid forcing a dependency on users who don't actually want to use ido with magit. Instead I will show a warning when these packages are not installed and the user attempts to use ido completion.
Ryan C. Thompson
@DarwinAwardWinner
That sounds reasonable. I would maybe set up magit-ido-completing-read so it tries to require idi-ubiquitous, and if it fails, issue a warning and either use ido-completing-read or stanard emacs completion instead.
Jonas Bernoulli
@tarsius
Ryan C. Thompson
@DarwinAwardWinner
Yes, it looks reasonable. I’m in the process of testing it now
Ryan C. Thompson
@DarwinAwardWinner
I think my testing is being stymyied by stale elc files
Giveev me a few minutes to reset
Jonas Bernoulli
@tarsius
You might want to (setq load-prefer-newer t) and give my auto-compile package a try.
Ryan C. Thompson
@DarwinAwardWinner
Yeah, I’ll check those out. I’m testing in a clean Emacs process right now, though
Jonas Bernoulli
@tarsius
Well, not now :-)
Ryan C. Thompson
@DarwinAwardWinner
Ok, it seems to be working correctly
I’ll let you know when I releasr ido-ubiquitous 3.0 so you can merge it.
Jonas Bernoulli
@tarsius
Okay, thanks. Any estimate?
Ryan C. Thompson
@DarwinAwardWinner
A few days
I need to dogfood the new version for a few days to see if it breaks anything
Jonas Bernoulli
@tarsius
Thanks again and have a nice <time of day>
Ryan C. Thompson
@DarwinAwardWinner
You too!
Ryan C. Thompson
@DarwinAwardWinner
Also, as for patching ido, everything in ido-completing-read+.el could probably be merged into ido, while the stuff still in ido-ubiquitous probably should remain separate.