Re: [Orgmode] [PATCH 0/2] Teach org-version to include git version information

[Bernt Hansen]

Nick Dokos <address@hidden> writes:

> Thanks! IIUC, there are then multiple forks in the decision tree:
>
>         o running on Windows or on something else?
>
>         o if on Windows, running native or under cygwin?
>
>         o what shell am I using? cmd.exe and bash under cygwin accept
>         different syntax - in particular, the GITDIR=foo solution of
>         setting env variables has no hope of working under cmd.exe -
>         is that correct?
>
> Sounds complicated. Maybe the best solution for now is to punt on
> the git information on Windows?
>
>     (when (not (member system-type '(ms-dos windows-nt cygwin)))
>           ...add git info...)
>
> What about system-type == "darwin"? Does Bernt's code work there?

Yes it'll be complicated.  Right now I think we should punt and just go
back to the old org-version reporting for windows until we can get a
better working solution since the current version breaks on windows.
I'll try to provide a patch for this.

Manish:

What value do you have for system-type?  I expect it says 'windows-nt'

If that's true then the problem is determining if
  - git is installed
    - what version?  Msysgit?  cygwin git?
  - what shells are available (since CMD.exe has little hope of working)
    - msysgit bash
    - cygwin shell
    - cmd (won't work if msysgit was installed with git in the msysgit
           bash shell only)
  - running some other shell (git-bash, cygwin) so the git executable is
    available and provide the right syntax for each to Do The Right
    Thing(tm)

Right now I'm inclined to skip the git-version magic if system-type is
windows-nt just to stop it from breaking on that platform.

-Bernt