Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

with dependency link to an hg repo, install fails to switch to correct revision (windows) #170

Closed
ghost opened this issue Mar 22, 2014 · 0 comments

Comments

@ghost
Copy link

ghost commented Mar 22, 2014

Originally reported by: ccanepa (Bitbucket: ccanepa, GitHub: ccanepa)


While testing the build for a new release of library cocos, http://code.google.com/p/los-cocos/ , I found a problem with a dependency, the pyglet library , http://code.google.com/p/pyglet/ .

With dependency link to hg repo, install fails to switch to correct revision.

It clones the repo, the dependency gets installed, but it is at the tip revision, not the one specified.

I think the relevant part of 'setup.py install' output is

#!

        Doing hg clone from https://code.google.com/p/pyglet to c:\docume~1\jugar\config
        ~1\temp\easy_install-icuiyz\pyglet@82693a20ece0
        Updating to 82693a20ece0
        No se esperaba >& en este momento.
        Processing pyglet@82693a20ece0  

'Updating to 82693a20ece0' seems to be emitted by setuptools before calling hg to do an hg update, ( yes, in setuptools.package_index.PackageIndex._download_hg , line 855 )

The line 'No se esperaba >& en este momento.' is comming from the OS, the meaning being 'unexpected >&'

In line 856 theres

#!python

 os.system("(cd %s && hg up -C -r %s >&-)" % (

Testing in a console standing in a pyglet's checkout I see

#!shell

hg up -C -r 82693a20ece0

works fine
whereas

#!shell

    D:\hg_externals\pyglet_dev>hg up -C -r 82693a20ece0 >&-
    No se esperaba >& en este momento.

So, the redirection in that os.system call is not working in Windows.

What is that ? >dev/null ? That would be >nul in classic Window, and $null in Windows's Powershell.

Would it be acceptable to replace the redirect with a '-q' (quiet) ?
That is, the line

#!python

           os.system("(cd %s && hg up -C -r %s >&-)" % (

changes to

#!python

            os.system("(cd %s && hg up -C -r %s -q)" % (

Details:
Running on windows xp
cocos package distribution built (from an export of revision r1357) both with py 2.6 and 3.3, both with setuptools 3.3; looks good.

The failure was observed when installed into a python 2.7 + setuptools 3.3

hg version is 2.9.1


@ghost ghost added major bug labels Mar 29, 2016
@jaraco jaraco closed this as completed in 5d163b5 Sep 13, 2017
jaraco added a commit that referenced this issue Aug 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

0 participants