{"action":"opened","issue":{"url":"https://api.github.com/repos/rodjek/librarian-puppet/issues/309","labels_url":"https://api.github.com/repos/rodjek/librarian-puppet/issues/309/labels{/name}","comments_url":"https://api.github.com/repos/rodjek/librarian-puppet/issues/309/comments","events_url":"https://api.github.com/repos/rodjek/librarian-puppet/issues/309/events","html_url":"https://github.com/rodjek/librarian-puppet/issues/309","id":88985754,"number":309,"title":"Can't specify a pull request revision in Puppetfile","user":{"login":"EslamElHusseiny","id":910187,"avatar_url":"https://avatars.githubusercontent.com/u/910187?v=3","gravatar_id":"","url":"https://api.github.com/users/EslamElHusseiny","html_url":"https://github.com/EslamElHusseiny","followers_url":"https://api.github.com/users/EslamElHusseiny/followers","following_url":"https://api.github.com/users/EslamElHusseiny/following{/other_user}","gists_url":"https://api.github.com/users/EslamElHusseiny/gists{/gist_id}","starred_url":"https://api.github.com/users/EslamElHusseiny/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/EslamElHusseiny/subscriptions","organizations_url":"https://api.github.com/users/EslamElHusseiny/orgs","repos_url":"https://api.github.com/users/EslamElHusseiny/repos","events_url":"https://api.github.com/users/EslamElHusseiny/events{/privacy}","received_events_url":"https://api.github.com/users/EslamElHusseiny/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-06-17T11:05:27Z","updated_at":"2015-06-17T11:05:27Z","closed_at":null,"body":"I'm trying to specify a certain pull request revision in my Puppetfile as following : \r\n> mod \"java\",\r\n>  :git => \"https://github.com/EslamElHusseiny/puppet-java.git\",\r\n>  :ref => \"a275d6aa7329868f0d7c6c770a95d6db514e0602\r\n\r\nbut actually the output of librarian-puppet install is : \r\n> Could not checkout https://github.com/EslamElHusseiny/puppet-java.git: fatal: ambiguous argument 'a275d6aa7329868f0d7c6c770a95d6db514e0602^{commit}': unknown revision or path not in the working tree.\r\n Use '--' to separate paths from revisions, like this:\r\n 'git <command> [<revision>...] -- [<file>...]'\r\n\r\nHowever I've checked that this revision exists as following : \r\n> git rev-parse a275d6aa7329868f0d7c6c770a95d6db514e0602^{commit}\r\na275d6aa7329868f0d7c6c770a95d6db514e0602\r\n\r\nalso as following : \r\n> git show-ref | grep a275d6aa7329868f0d7c6c770a95d6db514e0602\r\na275d6aa7329868f0d7c6c770a95d6db514e0602 refs/remotes/origin/pr/23/merge\r\n\r\nI'm using ruby version : \r\n> ruby -v \r\nruby 1.9.3p484 (2013-11-22 revision 43786) [x86_64-linux]\r\n\r\nand the distribution is : \r\n> lsb_release -a\r\nNo LSB modules are available.\r\nDistributor ID:\tUbuntu\r\nDescription:\tUbuntu 14.04.2 LTS\r\nRelease:\t14.04\r\nCodename:\ttrusty\r\n\r\nand \r\n> librarian-puppet install --verbose \r\n[Librarian] Ruby Version: 1.9.3\r\n[Librarian] Ruby Platform: x86_64-linux\r\n[Librarian] Rubygems Version: 1.8.23\r\n[Librarian] Librarian Version: 0.6.3\r\n[Librarian] Librarian Adapter: puppet\r\n[Librarian] Librarian Adapter Version: 2.2.0\r\n[Librarian] Project: /home/vagrant/workspace/PuppetLint/kitchenci\r\n[Librarian] Specfile: Puppetfile\r\n[Librarian] Lockfile: Puppetfile.lock\r\n[Librarian] Git: /usr/bin/git\r\n[Librarian] Git Version: 1.9.1\r\n[Librarian] Git Environment Variables:\r\n[Librarian]   (empty)\r\n[Librarian] Pre-Cached Sources:\r\n[Librarian] Post-Cached Sources:\r\n[Librarian]   [:forge, \"https://forgeapi.puppetlabs.com\", {}]\r\n[Librarian]   [:git, \"https://github.com/EslamElHusseiny/puppet-java.git\", {:ref=>\"a275d6aa7329868f0d7c6c770a95d6db514e0602\"}]\r\n[Librarian] Resolving maestrodev-wget (>= 1.3.2) <https://forgeapi.puppetlabs.com>\r\n[Librarian]   Checking manifests\r\n[Librarian]   Module maestrodev-wget found versions: 1.7.0, 1.6.0, 1.5.7, 1.5.6, 1.5.5, 1.5.4, 1.5.3, 1.5.2, 1.5.1, 1.5.0, 1.4.5, 1.4.4, 1.4.3, 1.4.2, 1.4.1, 1.4.0, 1.3.2, 1.3.1, 1.3.0, 1.2.3, 1.2.2, 1.2.1, 1.2.0, 1.1.0, 1.0.0, 0.0.1\r\n[Librarian]     Checking maestrodev-wget/1.7.0 <https://forgeapi.puppetlabs.com>\r\n[Librarian]       Resolved maestrodev-wget (>= 1.3.2) <https://forgeapi.puppetlabs.com> at maestrodev-wget/1.7.0 <https://forgeapi.puppetlabs.com>\r\n[Librarian]   Resolved maestrodev-wget (>= 1.3.2) <https://forgeapi.puppetlabs.com>\r\n[Librarian] Resolving java (>= 0) <https://github.com/EslamElHusseiny/puppet-java.git#a275d6aa7329868f0d7c6c770a95d6db514e0602>\r\n[Librarian]   Checking manifests\r\n[Librarian]     --- No output\r\n[Librarian]     --- No output\r\n[Librarian]     --- No output\r\n[Librarian]     --- No output\r\n[Librarian]     --> origin\r\n[Librarian]     -->   origin/HEAD -> origin/master\r\n[Librarian]     -->   origin/master\r\n[Librarian]     --- Exited with pid 24984 exit 128\r\n[Librarian]     --> fatal: ambiguous argument 'a275d6aa7329868f0d7c6c770a95d6db514e0602^{commit}': unknown revision or path not in the working tree.\r\n[Librarian]     --> Use '--' to separate paths from revisions, like this:\r\n[Librarian]     --> 'git <command> [<revision>...] -- [<file>...]'\r\nCould not checkout https://github.com/EslamElHusseiny/puppet-java.git: fatal: ambiguous argument 'a275d6aa7329868f0d7c6c770a95d6db514e0602^{commit}': unknown revision or path not in the working tree.\r\nUse '--' to separate paths from revisions, like this:\r\n'git <command> [<revision>...] -- [<file>...]'\r\n/var/lib/gems/1.9.1/gems/librarian-puppet-2.2.0/lib/librarian/puppet/source/git.rb:37:in `rescue in cache!'\r\n/var/lib/gems/1.9.1/gems/librarian-puppet-2.2.0/lib/librarian/puppet/source/git.rb:34:in `cache!'\r\n/var/lib/gems/1.9.1/gems/librarian-puppet-2.2.0/lib/librarian/puppet/source/local.rb:32:in `fetch_version'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/manifest.rb:93:in `fetch_version!'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/manifest.rb:85:in `fetched_version'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/manifest.rb:24:in `version'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/manifest.rb:20:in `to_s'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:191:in `block in scope_checking_manifest'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:236:in `block in debug'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/logger.rb:37:in `block in debug'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/ui.rb:32:in `debug'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/logger.rb:37:in `debug'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:236:in `debug'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:191:in `scope_checking_manifest'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:160:in `block (2 levels) in resolving_dependency_map_find_manifests'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:218:in `block in map_find'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:217:in `each'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:217:in `map_find'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:159:in `block in resolving_dependency_map_find_manifests'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:172:in `block (2 levels) in scope_resolving_dependency'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:186:in `block in scope_checking_manifests'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:230:in `scope'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:185:in `scope_checking_manifests'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:171:in `block in scope_resolving_dependency'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:230:in `scope'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:170:in `scope_resolving_dependency'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:158:in `resolving_dependency_map_find_manifests'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:65:in `do_resolve'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver/implementation.rb:50:in `resolve'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/resolver.rb:23:in `resolve'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/action/resolve.rb:31:in `run'\r\n/var/lib/gems/1.9.1/gems/librarian-puppet-2.2.0/lib/librarian/puppet/action/resolve.rb:10:in `run'\r\n/var/lib/gems/1.9.1/gems/librarian-puppet-2.2.0/lib/librarian/puppet/cli.rb:104:in `resolve!'\r\n/var/lib/gems/1.9.1/gems/librarian-puppet-2.2.0/lib/librarian/puppet/cli.rb:68:in `install'\r\n/var/lib/gems/1.9.1/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'\r\n/var/lib/gems/1.9.1/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'\r\n/var/lib/gems/1.9.1/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'\r\n/var/lib/gems/1.9.1/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/cli.rb:26:in `block (2 levels) in bin!'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/cli.rb:31:in `returning_status'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/cli.rb:26:in `block in bin!'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/cli.rb:47:in `with_environment'\r\n/var/lib/gems/1.9.1/gems/librarianp-0.6.3/lib/librarian/cli.rb:26:in `bin!'\r\n/var/lib/gems/1.9.1/gems/librarian-puppet-2.2.0/bin/librarian-puppet:7:in `<top (required)>'\r\n/usr/local/bin/librarian-puppet:23:in `load'\r\n/usr/local/bin/librarian-puppet:23:in `<main>'\r\n\r\nand the Puppetfile content is : \r\n> #!/usr/bin/env ruby\r\n> #^syntax detection\r\n\r\n> forge \"https://forgeapi.puppetlabs.com\"\r\nmodulefile\r\nmod \"java\",\r\n  :git => \"https://github.com/EslamElHusseiny/puppet-java.git\",\r\n  :ref => \"a275d6aa7329868f0d7c6c770a95d6db514e0602\"\r\n\r\n\r\nand the Modulefile content also is : \r\n> name 'puppet-java'\r\nversion '0.0.6'\r\nsource 'git@github.com:EslamElHusseiny/puppet-java.git'\r\nauthor 'Cake Team'\r\nlicense 'MIT License'\r\nsummary 'Java Puppet Module'\r\ndescription 'Java Puppet Module'\r\nproject_page 'https://github.com/EslamElHusseiny/puppet-java'\r\ndependency 'maestrodev/wget', '>= 1.3.2'\r\n"}}