Twenga / twgit

Twgit is a free and open source assisting tools for managing features, hotfixes and releases on Git repositories. It provides simple, high-level commands to adopt the branching model describes in our documentation. Supported OS: Debian/Ubuntu Linux, Mac OS X 10.8 (Mountain Lion) or lower.
http://twgit.twenga.com/
Other
111 stars 34 forks source link

Curl as failover of wget for connectors #146

Closed ltoussaint closed 9 years ago

ltoussaint commented 10 years ago

OSX does not have wget installed by default. Allow connectors to use curl as failover.

geoffroy-aubry commented 10 years ago

Don't forget to mention new Gitlab connector in README/CHANGELOG because no feature was dedicated.

geoffroy-aubry commented 10 years ago

Do python version of gitlab connector is working?

ltoussaint commented 10 years ago

This connector only work with PHP. With python it will return an empty string.

geoffroy-aubry commented 10 years ago

OK. Hmm… so why give priority to pyhton language in feature_subject_gitlab.sh, line 49? It wouldn't be better if python version doesn't work to simply remove it and throw an error?

ltoussaint commented 10 years ago

The chosen language always was PHP if installed. So the code with python was never call.

But to be compatible with the most OS, I did the python version. So it should be ok now.

geoffroy-aubry commented 10 years ago

"The chosen language always was PHP if installed.": sorry, but the code below seems say the opposite. Currently, in each connector, the chosen language is PHP if and only if Python is not installed. ^^

But if your Python version is working, it's fine :-)

# Python or PHP ?
language='?'
which python 1>/dev/null 2>&1
if [ $? -eq 0 ]; then
    language='python'
else
    which php 1>/dev/null 2>&1
    if [ $? -eq 0 ]; then
        language='php'
    fi
fi
ltoussaint commented 10 years ago

Yes, this code was added when I did the python version ;-)

geoffroy-aubry commented 10 years ago

OK, thx