On Sun, Aug 11, 2019 at 07:05:00AM +0200, Eric Auge wrote: > Hello, > > attached is the updated port for alacritty including a "pre-configure" > target data size check taken from the lang/rust port. > ok?
I would prefer the ulimit check to happen before the dependencies are downloaded, but that will need a suggestion from somebody who knows the cargo module better than me. I'm not sure the cargo module needs such a check by default, for instance, ripgrep and exa do just fine with the defaults. I am not going to use this port, as it is quite CPU hungry (~10-13% when just idling at the shell prompt) and for some reason it lstats its config file more than 10 times a second. Nevertheless, the port looks good to me now and if more experienced porters think this is ok portswise, I'm ok with importing this -- I certainly would not want to build this on a slower machine (it takes a whopping 45 minutes to build on my x280).
