On Wed Mar 6, 2024 at 11:17 AM EST, Omar Polo wrote:
> I've committed the update with devel/xdg-utils moved as RUN_DEPENDS.
>
> As sthen explained, LIB_DEPENDS is only for ports that provides shared
> libraries that this port links to.  Instead, devel/xdg-utils only
> provides a command that decker needs at runtime, so it has to be a
> RUN_DEPENDS.
>
> /usr/ports/infrastructure/bin/portcheck will caught these mistakes, it's
> handy to run it after building the port to catch small issues like this
> one.  (tip: it's handy to add /usr/ports/infrastructe/bin to $PATH)
>
> Also, while with CVS you can't (easily) add a directory unless you have
> write access, you can still `cvs add' new files.  This is better than
> attaching new files because it's easier to apply them.
>
> `cvs -q -n up -d' will tell you the state of the directory, with ?
> entries being files unknown to CVS, R files marked as removed with
> `cvs rm', A files added, M files modified, etc.
>
> I'm attaching the diff that I've actually committed.
>
> Thanks!

Thank you so much for the help, Omar! I'll do my best to remember these for my
next patches.

-- 
Jag Talon (he/him)

https://jagtalon.net/
https://weirder.earth/@jag

• 2F17E7825E755F08.asc (6K)