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

git: get_head_branch incorrect when branch name contains a slash #6672

Closed
thpani opened this issue Mar 25, 2014 · 1 comment
Closed

git: get_head_branch incorrect when branch name contains a slash #6672

thpani opened this issue Mar 25, 2014 · 1 comment
Labels
bug This issue/PR relates to a bug.

Comments

@thpani
Copy link

thpani commented Mar 25, 2014

Issue Type:

Bug Report

Ansible Version:

1.5.3

Environment:

OS X 10.9.2

Summary:

get_head_branch() fails to correctly compute the HEAD branch if the branch name contains a slash. This is caused by splitting on '/' and indexing [-1] on https://github.com/ansible/ansible/blob/devel/library/source_control/git#L350

Steps To Reproduce:

Run from this playbook:

- name: checkout llvm
  git: repo=https://github.com/thpani/llvm.git
       dest=/home/vagrant/llvm/

Checkout fails because branch is computed as "release_33_nullsucc" instead of "patches/release_33_nullsucc".

Expected Results:

get_head_branch() computes the correct branch.

I.e. behave as if the above example were

- name: checkout llvm
  git: repo=https://github.com/thpani/llvm.git
       dest=/home/vagrant/llvm/
       version=patches/release_33_nullsucc
Actual Results:
TASK: [checkout llvm] *********************************************************
failed: [default] => {"failed": true}
msg: Failed to checkout branch release_33_nullsucc
@mpdehaan mpdehaan added bug_report and removed P3 labels Mar 25, 2014
@mpdehaan mpdehaan added P5 and removed P4 labels Apr 17, 2014
@mpdehaan
Copy link
Contributor

Hi!

Thanks very much for your interest in Ansible. It sincerely means a lot to us.

On September 26, 2014, due to enormous levels of contribution to the project Ansible decided to reorganize module repos, making it easier
for developers to work on the project and for us to more easily manage new contributions and tickets.

We split modules from the main project off into two repos, http://github.com/ansible/ansible-modules-core and http://github.com/ansible/ansible-modules-extras

If you would still like this ticket attended to, and believe this problem or idea is still present in the latest version of Ansible (1.7.2) or the development branch, we will need your help in having it reopened in one of the two new repos, and instructions are provided below.

We apologize that we are not able to make this transition happen seamlessly, though this is a one-time change and your help is greatly appreciated --
this will greatly improve velocity going forward.

Both sets of modules will ship with Ansible, though they'll receive slightly different ticket handling.

To locate where a module lives between 'core' and 'extras'

Additionally, should you need more help with this, you can ask questions on:

Thank you very much!

@ansibot ansibot added bug This issue/PR relates to a bug. and removed bug_report labels Mar 6, 2018
@ansible ansible locked and limited conversation to collaborators Apr 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug This issue/PR relates to a bug.
Projects
None yet
Development

No branches or pull requests

3 participants