On 2026/10/16 18:29, David Uhden Collado wrote:
> Stuart Henderson wrote:
> > > > +WRKDIST =      ${WRKDIR}/${MODGO_MODNAME}@${MODGO_VERSION}
> > 
> > the idiom for this when you need to patch a go port is:
> > 
> > WRKDIST=     ${WRKSRC}
> > 
> > please don't add different versions
> 
> Thanks for the clarification. Do you know why WRKDIST doesn't point to the
> correct directory under WRKDIR in ports that use the Go and Cargo modules?

It does for Cargo. (and yes, I do know why for go).

> > This is not really a mechanism which promotes doing things safely in
> > the first place. The mechanism used by sudoedit (copy to tempfile with
> > privs, edit without privs, copy back with privs) is better (but I
> > suppose what upstream really want to do here is cope with the common
> > user mistake of editing a file without thinking about whether they can
> > write it or not beforehand). People can always switch to another term
> > and temporarily chown/chmod anyway...
> 
> Anyway, having hidden runtime dependencies is not a good thing. They need to
> be identified and eliminated.

we often leave deps that are only needed for certain functions as optional..

Reply via email to