mmstick / cargo-deb

A cargo subcommand that generates Debian packages from information in Cargo.toml
615 stars 50 forks source link

Asset source dir not recursively copied #90

Closed apiraino closed 5 years ago

apiraino commented 5 years ago

Hey,

first of all, thanks for this very useful cargo extension! :+1:

Now, something I don't fully understand. I would like to copy all migrations:

$ tree migrations/
migrations/
├── 2018-06-03-132600_create_users
│   ├── down.sql
│   └── up.sql
└── 2018-08-10-151755_create_doors
    ├── down.sql
    └── up.sql

2 directories, 4 files

This does not copy recursively:

assets = [
    ["migrations/", "migrations/", "755"],
]

This works:

assets = [
    ["migrations/*/*", "migrations/", "755"],
]

bug or feature? :-)

Thanks!

kornelski commented 5 years ago

It's sort-of a feature. We use a simple glob() that does a shallow directory scan.

apiraino commented 5 years ago

ok, I think that makes sense. The other way around would require more thought (f.e. someone may want a recursive scan but with exceptions).