Thanks for the first review. I’ve corrected the PREFIX and uploaded the files again. May I ask for another review?
---------------------------------------------------------------------- Hints: ---------------------------------------------------------------------- mdp-1.0.9-1.hint ---------------------------------------------------------------------- category: Utils requires: cygwin libncursesw10 sdesc: "A command-line based markdown presentation tool" ldesc: "A ncurses-based command-line presentation tool, which makes it easy to create slides using the popular markdown format." mdp-debuginfo-1.0.9-1.hint ---------------------------------------------------------------------- category: Debug requires: cygwin-debuginfo external-source: mdp sdesc: "Debug info for mdp" ldesc: "This package contains files necessary for debugging the mdp package with gdb." ---------------------------------------------------------------------- Packages: ---------------------------------------------------------------------- x86_64: ---------------------------------------------------------------------- http://raw.nixhub.de/cygwin/x86_64/mdp/mdp-1.0.9-1.tar.xz http://raw.nixhub.de/cygwin/x86_64/mdp/mdp-1.0.9-1-src.tar.xz http://raw.nixhub.de/cygwin/x86_64/mdp/mdp-1.0.9-1.hint http://raw.nixhub.de/cygwin/x86_64/mdp/mdp-debuginfo/mdp-debuginfo-1.0.9-1.tar.xz http://raw.nixhub.de/cygwin/x86_64/mdp/mdp-debuginfo/mdp-debuginfo-1.0.9-1.hint x86: ---------------------------------------------------------------------- http://raw.nixhub.de/cygwin/x86/mdp/mdp-1.0.9-1.tar.xz http://raw.nixhub.de/cygwin/x86/mdp/mdp-1.0.9-1-src.tar.xz http://raw.nixhub.de/cygwin/x86/mdp/mdp-1.0.9-1.hint http://raw.nixhub.de/cygwin/x86/mdp/mdp-debuginfo/mdp-debuginfo-1.0.9-1.tar.xz http://raw.nixhub.de/cygwin/x86/mdp/mdp-debuginfo/mdp-debuginfo-1.0.9-1.hint > On 9 Jan 2017, at 18:26, Jon Turney <jon.tur...@dronecode.org.uk> wrote: > > On 30/12/2016 16:11, Michael Göhler wrote: >> Hi, >> >> i’ve packaged mdp and would like to become the package maintainer for it. >> mdp is available at https://github.com/visit1985/mdp under GPLv3. >> >> Currently it is included in: >> * Arch Linux: https://aur.archlinux.org/packages/mdp-git/ >> * Debian: https://tracker.debian.org/pkg/mdp-src >> * FreeBSD: http://www.freshports.org/misc/mdp >> * OS-X: http://brewformulas.org/Mdp >> * Slackware: http://slackbuilds.org/apps/mdp/ >> * Ubuntu: https://launchpad.net/ubuntu/+source/mdp-src >> > > Thanks for this. > > It seems that these packages install mdp.exe into /usr/local/bin/. > per [1], this is not correct. > > I guess you need to investigate supplying the correct value of PREFIX to the > Makefile. > > Apart from that, looks good to me. > > [1] https://cygwin.com/packaging-package-files.html#package_contents > >