Hello, An update after some useful comments from a person (Alexis Hildebrandt) off list, as diff:
--- alacritty/Makefile
+++ alacritty/Makefile
@@ -7,9 +7,9 @@
DISTFILES += ${DISTNAME}${EXTRACT_SUFX}
CATEGORIES = x11
-MAINTAINER = Eric Auge <[email protected]>
+MAINTAINER = Eric Auge <[email protected]>
-GH_ACCOUNT = jwilm
+GH_ACCOUNT = alacritty
GH_PROJECT = alacritty
GH_TAGNAME = v0.9.0
@@ -49,6 +49,7 @@
${INSTALL_DATA_DIR} ${PREFIX}/share/examples/alacritty
${INSTALL_DATA} ${WRKSRC}/alacritty.yml \
${PREFIX}/share/examples/alacritty/alacritty.yml
+ tic -s -x -o ${PREFIX}/share/terminfo ${EXTRA_DIR}/alacritty.info
.include "crates.inc"
Please let me know if anything else should be changed.
Attached the updated package.
hth,
Cheers,
Eric.
On Sat, Oct 16, 2021 at 10:39 PM Eric Auge <[email protected]> wrote:
>
> ping.
>
> On Sat, Sep 11, 2021 at 2:18 PM Eric Auge <[email protected]> wrote:
> >
> > Hello,
> >
> > attached the updated the port (followed Stuart comments):
> > - made a patch for the live_config_reload default.
> > - updated DESCR and README according to comments & alacritty updated
> > self description.
> > - the xcursors-themes port does not seem necessary anymore.
> >
> > Did I miss anything?
> >
> > HTH,
> > Eric.
> >
> > On Fri, Sep 3, 2021 at 2:41 PM Stuart Henderson <[email protected]>
> > wrote:
> > >
> > > On 2021/09/03 16:31, Frederic Cambus wrote:
> > > > On Mon, Aug 12, 2019 at 07:39:23AM +0200, Theo Buehler wrote:
> > > > > On Sun, Aug 11, 2019 at 11:30:09PM +0100, Stuart Henderson wrote:
> > > > > > So the claimed advantage is that it's super fast, but actually it
> > > > > > uses way
> > > > > > more cpu than probably any other terminal emulator in the tree
> > > > > > except
> > > > > > cool-retro-term?
> > > > >
> > > > > Yes, it is blazing fast. matthieu's test of 'cat /etc/termcap'
> > > > > completes
> > > > > in under 0.1s while xterm uses 0.2-0.3s. It does feel way snappier
> > > > > than
> > > > > xterm in general.
> > > > >
> > > > > It looks like I made CPU use worse by about a factor of 2 due to using
> > > > > vm.malloc_conf=CFJ. Still, ~5% is way more than it should need.
> > > > >
> > > > > Another advantage is that contrary to xterm it is trivial to configure
> > > > > thanks to a self-documenting .yml config file.
> > > > >
> > > > > > Is this useful enough to be worth adding so much time to a bulk
> > > > > > build?
> > > > >
> > > > > I doubt it.
> > > > >
> > > > > > It will likely need to be amd64-only.
> > > > >
> > > > > Probably. Testing on other archs would need more time than I currently
> > > > > have.
> > > >
> > > > Resurrecting this as it seems the requirements are much more reasonable
> > > > nowadays, and I can build the latest version in 4 minutes on my Ryzen 5
> > > > PRO 2500U laptop:
> > > >
> > > > Finished release [optimized] target(s) in 4m 07s
> > > >
> > > > Here is a new tarball with the following changes:
> > > >
> > > > - Update to alacritty 0.9.0
> > > > - Move MODCARGO_CRATES directives into crates.inc.
> > > > - Remove the ulimit checks, as I was able to build the port as a regular
> > > > user with the default limits without running out of memory
> > >
> > > README needs either rewriting or removing. Some of the sentences don't
> > > parse. The configuration section says what the program does but doesn't
> > > really say what the user should do with that information.
> > >
> > > There's a conflict between what DESCR says ("It provides sane choices
> > > for defaults and requires no additional setup") and README: "Although
> > > the default configuration work on OpenBSD, some values are not optimal
> > > [...] This option leads alacritty to a non negligeable CPU overhead on
> > > OpenBSD". If it's bad enough to document then I would suggest patching
> > > the default setting for live_config_reload instead. And rather than
> > > telling the user "please install pkg_add xcursors-themes port" it would
> > > be better to just RUN_DEPENDS on it. If that's done then there isn't
> > > really any more need for the pkg-readme, the standard documentation
> > > should suffice.
> > >
alacritty.tar.gz
Description: application/gzip
