Re: WIP/Help wanted: use MULTI_PACKAGES for net/snowflake_proxy

2022-12-23 Thread Yifei Zhan
Many thanks to Struct and Omar for clearing my doubt around this :) I agree the following patch is a better idea, I tested caspar's tweaked diff and it builds/runs with no issue. > > So below is a diff with a slightly tweaked COMMENT. How about this? > This is OK caspar@ > > Caspar > > >

Re: WIP/Help wanted: use MULTI_PACKAGES for net/snowflake_proxy

2022-12-22 Thread Caspar Schutijser
On Thu, Dec 22, 2022 at 08:17:46AM +, Stuart Henderson wrote: > Don't overthink things though, there's really no need to split this > up, a single package is fine (and maybe preferable in this case, as it > stops a passive observer from figuring out whether you downloaded > the proxy or server

Re: WIP/Help wanted: use MULTI_PACKAGES for net/snowflake_proxy

2022-12-22 Thread Stuart Henderson
On 2022/12/22 09:03, Omar Polo wrote: > On 2022/12/22 00:27:59 +, Yifei Zhan wrote: > > MULTI_PACKAGES= -main -server -client > > > > ...cut... > > > > .if ${BUILD_PACKAGES:M-main} > > ALL_TARGET= ./proxy/ > > .endif > > > > .if ${BUILD_PACKAGES:M-client} > > ALL_TARGET= ./client/

Re: WIP/Help wanted: use MULTI_PACKAGES for net/snowflake_proxy

2022-12-22 Thread Omar Polo
On 2022/12/22 00:27:59 +, Yifei Zhan wrote: > Here is an attempt to convert net/snowflake_proxy into subpackages. > > snowflake_proxy has 3 components, correspond to 3 subpackages: > > - standalone server, tor not required (The only currently packaged components) > - serverside managed