{"action":"created","issue":{"url":"https://api.github.com/repos/ocaml/opam/issues/1422","repository_url":"https://api.github.com/repos/ocaml/opam","labels_url":"https://api.github.com/repos/ocaml/opam/issues/1422/labels{/name}","comments_url":"https://api.github.com/repos/ocaml/opam/issues/1422/comments","events_url":"https://api.github.com/repos/ocaml/opam/issues/1422/events","html_url":"https://github.com/ocaml/opam/issues/1422","id":34384627,"number":1422,"title":"Handle Git submodules","user":{"login":"NicolasT","id":102605,"avatar_url":"https://avatars.githubusercontent.com/u/102605?v=3","gravatar_id":"","url":"https://api.github.com/users/NicolasT","html_url":"https://github.com/NicolasT","followers_url":"https://api.github.com/users/NicolasT/followers","following_url":"https://api.github.com/users/NicolasT/following{/other_user}","gists_url":"https://api.github.com/users/NicolasT/gists{/gist_id}","starred_url":"https://api.github.com/users/NicolasT/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NicolasT/subscriptions","organizations_url":"https://api.github.com/users/NicolasT/orgs","repos_url":"https://api.github.com/users/NicolasT/repos","events_url":"https://api.github.com/users/NicolasT/events{/privacy}","received_events_url":"https://api.github.com/users/NicolasT/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ocaml/opam/labels/Feature%20Wish","name":"Feature Wish","color":"02d7e1"}],"state":"closed","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/ocaml/opam/milestones/7","html_url":"https://github.com/ocaml/opam/milestones/Next","labels_url":"https://api.github.com/repos/ocaml/opam/milestones/7/labels","id":137072,"number":7,"title":"Next","description":"These issues have been reviewed, but didn't cut it for the currently planned release. They'll be rescheduled as we release.","creator":{"login":"samoht","id":103693,"avatar_url":"https://avatars.githubusercontent.com/u/103693?v=3","gravatar_id":"","url":"https://api.github.com/users/samoht","html_url":"https://github.com/samoht","followers_url":"https://api.github.com/users/samoht/followers","following_url":"https://api.github.com/users/samoht/following{/other_user}","gists_url":"https://api.github.com/users/samoht/gists{/gist_id}","starred_url":"https://api.github.com/users/samoht/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/samoht/subscriptions","organizations_url":"https://api.github.com/users/samoht/orgs","repos_url":"https://api.github.com/users/samoht/repos","events_url":"https://api.github.com/users/samoht/events{/privacy}","received_events_url":"https://api.github.com/users/samoht/received_events","type":"User","site_admin":false},"open_issues":78,"closed_issues":53,"state":"open","created_at":"2012-06-26T09:09:10Z","updated_at":"2016-06-01T10:14:59Z","due_on":null,"closed_at":null},"comments":8,"created_at":"2014-05-27T15:38:41Z","updated_at":"2016-06-01T10:14:59Z","closed_at":"2016-06-01T10:14:59Z","body":"We have some sources for which we provide Opam package descriptions in our `devel` repository, which use a `git` tree as `url`.\r\nWhilst this works just fine, a recent change causes trouble: instead of containing some third-party sources in the repository as a tarball, we switched to using a *git submodule*. Using a submodule requires some extra care running `git submodule update --init` after cloning the repository, or running `git clone` with the `--recursive` flag.\r\nSince Opam doesn't handle submodules, initially the build of our package failed (since files were missing). I added `git submodule update --init` to the build steps (https://github.com/Incubaid/opam-repository-devel/pull/6), which now works, but sub-optimally: I didn't dig into this yet, but it looks like the repository which is checked out in `packages.dev/camltc` is copied into a `build` folder, in which the actual building is done.\r\nSince the submodule content isn't in the `packages.dev/camltc` folder, this results in the submodule to be re-cloned for every build, which is rather undesired.\r\n\r\nI think the Opam Git support should initialize submodules when cloning a repository to `packages.dev`, and update submodules after every update/reset action, so the submodule contents are copied into the `build` folder as expected, and no `git` commands are required in the Opam package build script."},"comment":{"url":"https://api.github.com/repos/ocaml/opam/issues/comments/222951257","html_url":"https://github.com/ocaml/opam/issues/1422#issuecomment-222951257","issue_url":"https://api.github.com/repos/ocaml/opam/issues/1422","id":222951257,"user":{"login":"AltGr","id":807966,"avatar_url":"https://avatars.githubusercontent.com/u/807966?v=3","gravatar_id":"","url":"https://api.github.com/users/AltGr","html_url":"https://github.com/AltGr","followers_url":"https://api.github.com/users/AltGr/followers","following_url":"https://api.github.com/users/AltGr/following{/other_user}","gists_url":"https://api.github.com/users/AltGr/gists{/gist_id}","starred_url":"https://api.github.com/users/AltGr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AltGr/subscriptions","organizations_url":"https://api.github.com/users/AltGr/orgs","repos_url":"https://api.github.com/users/AltGr/repos","events_url":"https://api.github.com/users/AltGr/events{/privacy}","received_events_url":"https://api.github.com/users/AltGr/received_events","type":"User","site_admin":false},"created_at":"2016-06-01T10:14:59Z","updated_at":"2016-06-01T10:14:59Z","body":"opam remotes based on git will now automatically fetch submodules too, so this has been implemented. Thanks for the feedback!"}}
{"actor":{"display_login":"AltGr"}}