Changeset View
Standalone View
package.yml
- This file was added.
| name : broot | |||||
| version : 0.13.5 | |||||
| release : 1 | |||||
| source : | |||||
| - https://github.com/Canop/broot/archive/v0.13.5b.tar.gz : 8a37d66a30f15c64db5ffeded1f2b6d808bd80261d7e8d97e6288cfd1db621dd | |||||
| license : MIT | |||||
| component : system.utils | |||||
| summary : A new way to see and navigate directory trees | |||||
| description: | | |||||
| A new way to see and navigate directory trees using cd, tree, and fuzzy search. | |||||
| networking : yes | |||||
DataDrake: Should be a list format, even for one item. | |||||
| builddeps : | |||||
| cargo | |||||
Done Inline Actionsdescription should go after summary, not after builddeps. JoshStrobl: description should go after summary, not after builddeps. | |||||
| build : | | |||||
Done Inline ActionsDid you mean "setup"? "prep" doesn't do anything. DataDrake: Did you mean "setup"? "prep" doesn't do anything. | |||||
| cargo build --release --locked | |||||
| install : | | |||||
| install -Dm00755 target/release/broot $installdir/usr/bin/broot | |||||
| install -Dm00644 target/release/build/broot-d686648d24283199/out/_br $installdir/usr/share/zsh/site-functions/_br | |||||
Done Inline ActionsAccording to https://github.com/Canop/broot/blob/master/build.rs they are also building shell completions and I see a man page dir in the project's root. We should ideally be installing what shell completions they do have:
manpage location will be determined by the section they are expecting, probably 1 for general commands, so it should be installed to /usr/share/man/man1/ and be sure it has .1 appended to the end of the man page file name. JoshStrobl: According to https://github.com/Canop/broot/blob/master/build.rs they are also building shell… | |||||
| install -Dm00644 target/release/build/broot-d686648d24283199/out/_broot $installdir/usr/share/zsh/site-functions/_broot | |||||
Done Inline ActionsSeems like a fair bit of redundancy with how you're doing completions installation. You could more easily accomplish all of this using a pushd into the target out directory and loop a list of the completions or the directories and install from there. Otherwise it's a good start. JoshStrobl: Seems like a fair bit of redundancy with how you're doing completions installation. You could… | |||||
| install -Dm00644 target/release/build/broot-d686648d24283199/out/br.fish $installdir/usr/share/fish/completions/br.fish | |||||
Done Inline ActionsShould just be: install -Dm00755 target/release/broot $installdir/usr/bin/broot DataDrake: Should just be:
```
install -Dm00755 target/release/broot $installdir/usr/bin/broot
``` | |||||
| install -Dm00644 target/release/build/broot-d686648d24283199/out/broot.fish $installdir/usr/share/fish/completions/broot.fish | |||||
Done Inline ActionsWe don't ship license files unless upstream does it of their own volition. DataDrake: We don't ship license files unless upstream does it of their own volition. | |||||
| install -Dm00644 target/release/build/broot-d686648d24283199/out/br.bash $installdir/usr/share/bash-completion/completions/br.bash | |||||
| install -Dm00644 target/release/build/broot-d686648d24283199/out/broot.bash $installdir/usr/share/bash-completion/completions/broot.bash | |||||
Should be a list format, even for one item.