On 1/9/14, 6:26 PM, Anil Madhavapeddy wrote:
> In packages/acpc/acpc.1.0/opam:
>
>> @@ -0,0 +1,19 @@
>> +opam-version: "1"
>> +maintainer: "berenger@riken.jp"
>> +authors: ["Francois Berenger"]
>> +homepage: "https://github.com/UnixJunkie/ACPC"
>> +license: "BSD"
>> +build: [
>> +  ["obuild" "configure"]
>> +  ["obuild" "build"]
>> +  ["cp" "dist/build/acpc/acpc" "dist/build/acpc_big/acpc_big" "dist/build/acpc_mol2tool/acpc_mol2too" bin]
>
> It's better to create a |acpc.install| file rather than manually copy
> them to the bindir, and gives OPAM more useful metadata. Any reason not
> to include that instead of this?

I did not know about this opam feature (X.install).

I guess OPAM will be able to install and uninstall automatically
in case I use a .install file?

I'll try to submit another pull request tomorrow then.

> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ocaml/opam-repository/pull/1547/files#r8750886>.
>