Merge pull request #1 from pedrocarmona/add-version-variable

Add version variable

Author: pedrocarmona

.travis.yml

@@ -22,15 +22,21 @@ script:
   - "ansible-playbook -i tests/inventory tests/$SITE --syntax-check"
 
   # Run the role/playbook with ansible-playbook.
-  - "ansible-playbook -i tests/inventory tests/$SITE --connection=local --sudo"
+  - "ansible-playbook -i tests/inventory tests/$SITE --connection=local"
 
   # Run the role/playbook again, checking to make sure it's idempotent.
   - >
-    ansible-playbook -i tests/inventory tests/$SITE --connection=local --sudo
-    | grep -q 'changed=1.*failed=0'
+    ansible-playbook -i tests/inventory tests/$SITE --connection=local
+    | grep -q 'changed=0.*failed=0'
     && (echo 'Idempotence test: pass' && exit 0)
     || (echo 'Idempotence test: fail' && exit 1)
 
+  - >
+    git lfs version | awk -F' ' '{print $1}' | awk -F'/' '{print $2}'
+    | grep -q '1.1.0'
+    && (echo 'version test: pass' && exit 0)
+    || (echo 'version test: fail' && exit 1)
+
   # Make sure git is installed and at the correct version.
   - which git-lfs
   - git lfs env

README.md

@@ -19,26 +19,6 @@ Sponsored by [BidMath](http://bidmath.com/)
 ```
 
 
-## Usage notes
-
-Added the git_lfs_install_vars variable to defaults, in order to provide the ability to not download files in LFS when you clone the repo. Currently is faster to clone a repo and then use git lfs pull, because git lfs pull has better implementation on download:
-
-```yaml
-
-- hosts: servers
-  roles:
-    - { role: pedrocarmona.github-git-lfs }
-  tasks:
-    - git: >
-      repo=git://foosball.example.org/path/to/repo.git
-      dest=repodir/
-
-    - command: git lfs pull
-      args:
-        chdir: repodir/
-```
-
-
 ## License
 
 MIT / BSD

defaults/main.yml

@@ -1,2 +1,2 @@
 ---
-git_lfs_install_vars: --skip-smudge
+git_lfs_version: 1.1.0

tasks/debian.yml

@@ -19,6 +19,6 @@
 
 - name: Ensure github git-lfs is installed (Debian)
   apt:
-    name=git-lfs
-    state=latest
+    name=git-lfs={{ git_lfs_version }}
+    state=present
     update_cache=yes

tasks/main.yml

@@ -4,7 +4,3 @@
 
 - include: debian.yml
   when: ansible_os_family == 'Debian'
-
-- name:  Git lfs install
-  sudo: false
-  command: git lfs install {{ git_lfs_install_vars }}

tasks/redhat.yml

@@ -15,7 +15,7 @@
 
 - name: Ensure github git-lfs is installed (RedHat)
   yum:
-    name=git-lfs
-    state=latest
+    name=git-lfs-{{ git_lfs_version }}
+    state=present
     update_cache=yes
     enablerepo=github_git-lfs

tests/test-install-package.yml

@@ -1,5 +1,8 @@
 ---
 - hosts: localhost
-  remote_user: root
+  sudo: true
+  vars:
+    user: deploy
+    group: deploy
   roles:
     - github-git-lfs