Page MenuHomeFreeBSD

zirias (Felix Palmen)
User

Projects

User does not belong to any projects.

User Details

User Since
May 6 2021, 8:57 AM (65 w, 3 d)

Recent Activity

Yesterday

zirias requested review of D36060: Update Ronald Klop's email address in contributors.
Sun, Aug 7, 11:18 AM
zirias closed D36059: comms/opencbm: Fix PORTSCOUT limit.
Sun, Aug 7, 9:14 AM
zirias committed R11:42f8c0bf850d: comms/opencbm: Fix PORTSCOUT limit (authored by zirias).
comms/opencbm: Fix PORTSCOUT limit
Sun, Aug 7, 9:14 AM
zirias requested review of D36059: comms/opencbm: Fix PORTSCOUT limit.
Sun, Aug 7, 9:10 AM
zirias committed R11:59a9ef205ec5: multimedia/makemkv: Update to 1.17.1 (authored by zirias).
multimedia/makemkv: Update to 1.17.1
Sun, Aug 7, 7:09 AM
zirias committed R11:370d7900f0c9: multimedia/makemkv: Update to 1.17.0 (authored by zirias).
multimedia/makemkv: Update to 1.17.0
Sun, Aug 7, 7:09 AM
zirias committed R11:88ac87ec1940: multimedia/makemkv: Add support for aarch64 (authored by zirias).
multimedia/makemkv: Add support for aarch64
Sun, Aug 7, 7:09 AM
zirias closed D36039: multimedia/makemkv: Update to 1.17.1.
Sun, Aug 7, 6:52 AM
zirias committed R11:28f74df622d0: multimedia/makemkv: Update to 1.17.1 (authored by zirias).
multimedia/makemkv: Update to 1.17.1
Sun, Aug 7, 6:52 AM

Sat, Aug 6

zirias updated the diff for D36039: multimedia/makemkv: Update to 1.17.1.

Make use of USE_C[XX]STD.

Sat, Aug 6, 5:36 PM
zirias added inline comments to D36039: multimedia/makemkv: Update to 1.17.1.
Sat, Aug 6, 5:29 PM
zirias updated the diff for D36039: multimedia/makemkv: Update to 1.17.1.

Replace PORTVERSION with DISTVERSION.

Sat, Aug 6, 7:51 AM
zirias updated the diff for D36039: multimedia/makemkv: Update to 1.17.1.

Rename FDKAAC option to match multimedia/ffmpeg, disable it by default.

Sat, Aug 6, 7:23 AM
zirias added inline comments to D36039: multimedia/makemkv: Update to 1.17.1.
Sat, Aug 6, 7:12 AM
zirias added inline comments to D36039: multimedia/makemkv: Update to 1.17.1.
Sat, Aug 6, 6:56 AM
zirias added inline comments to D36039: multimedia/makemkv: Update to 1.17.1.
Sat, Aug 6, 6:49 AM

Fri, Aug 5

zirias updated the diff for D36039: multimedia/makemkv: Update to 1.17.1.

Add two minor improvements.

Fri, Aug 5, 10:11 AM

Thu, Aug 4

zirias requested review of D36039: multimedia/makemkv: Update to 1.17.1.
Thu, Aug 4, 7:14 AM

Wed, Aug 3

zirias closed D36031: comms/opencbm: Update to 0.4.99.104.
Wed, Aug 3, 7:21 PM
zirias committed R11:0a90d3d9f966: comms/opencbm: Update to 0.4.99.104 (authored by zirias).
comms/opencbm: Update to 0.4.99.104
Wed, Aug 3, 7:21 PM
zirias updated the diff for D36031: comms/opencbm: Update to 0.4.99.104.

Replace sed with upstream patch.

Wed, Aug 3, 10:20 AM
zirias updated the diff for D36031: comms/opencbm: Update to 0.4.99.104.

Extra newline as suggested.

Wed, Aug 3, 10:01 AM
zirias added inline comments to D36031: comms/opencbm: Update to 0.4.99.104.
Wed, Aug 3, 9:55 AM
zirias requested review of D36031: comms/opencbm: Update to 0.4.99.104.
Wed, Aug 3, 9:29 AM
zirias closed D36029: maintainer: Update to FreeBSD address.
Wed, Aug 3, 9:01 AM
zirias committed R11:0ac86ecea3d4: maintainer: Update to FreeBSD address (authored by zirias).
maintainer: Update to FreeBSD address
Wed, Aug 3, 9:01 AM
zirias requested review of D36029: maintainer: Update to FreeBSD address.
Wed, Aug 3, 8:03 AM
zirias committed rGe21dc161b346: Add me as ports committer, update mentor/mentee (authored by zirias).
Add me as ports committer, update mentor/mentee
Wed, Aug 3, 7:42 AM
zirias closed D36014: Add me as ports committer, update mentor/mentee.
Wed, Aug 3, 7:42 AM
zirias closed D36011: New committer (ports): Felix Palmen (zirias).
Wed, Aug 3, 7:29 AM
zirias committed R9:2d79e95decfc: New committer (ports): Felix Palmen (zirias) (authored by zirias).
New committer (ports): Felix Palmen (zirias)
Wed, Aug 3, 7:29 AM

Tue, Aug 2

zirias requested review of D36014: Add me as ports committer, update mentor/mentee.
Tue, Aug 2, 3:25 PM
zirias requested review of D36011: New committer (ports): Felix Palmen (zirias).
Tue, Aug 2, 12:53 PM
pi renamed zirias from felix_palmen-it.de to zirias.
Tue, Aug 2, 12:06 PM

Sat, Jul 30

zirias added a comment to D35348: x11/xscreensaver: Update to 6.04.
In D35348#817314, @cy wrote:

You don't need to articulate each adjustment. You need to mention that adjustments outside of the primary change will be made.

Sat, Jul 30, 4:32 PM

Fri, Jul 29

zirias added a comment to D35348: x11/xscreensaver: Update to 6.04.

Seriously, I think it would be pretty ridiculous to make one single line break an extra commit. But I'll happily mention it in the commit message of course ;)

Fri, Jul 29, 9:07 PM
zirias updated the diff for D35348: x11/xscreensaver: Update to 6.04.

rebase after gdk-pixbuf change

Fri, Jul 29, 5:12 PM
zirias updated the diff for D35348: x11/xscreensaver: Update to 6.04.

rebase after created-by removal

Fri, Jul 29, 5:02 PM

Mon, Jul 25

zirias added a comment to D35805: net/linphone: Update to 4.4.8.

Ping - anyone working on the one-liner to build Linphone w/GIT installed?

Mon, Jul 25, 4:03 PM
zirias added a comment to D35904: textproc/linux-c7-expat-devel: Add new port.

For some more background:

Mon, Jul 25, 11:15 AM
zirias added a reviewer for D35904: textproc/linux-c7-expat-devel: Add new port: emulation.
Mon, Jul 25, 10:30 AM
zirias added a reviewer for D35903: security/linux-c7-openssl-devel: Add new port: emulation.
Mon, Jul 25, 10:29 AM
zirias added a reviewer for D35902: devel/linux-c7-zlib-devel: Add new port: emulation.
Mon, Jul 25, 10:29 AM
zirias updated the diff for D35904: textproc/linux-c7-expat-devel: Add new port.

Add missing SUBDIR.

Mon, Jul 25, 9:28 AM
zirias updated the diff for D35903: security/linux-c7-openssl-devel: Add new port.

Add missing SUBDIR.

Mon, Jul 25, 9:27 AM
zirias updated the diff for D35902: devel/linux-c7-zlib-devel: Add new port.

Add missing SUBDIR.

Mon, Jul 25, 9:27 AM
zirias added a comment to D35904: textproc/linux-c7-expat-devel: Add new port.

These new ports would enable me to get rid of several dirty hacks in multimedia/makemkv.

Mon, Jul 25, 9:05 AM
zirias requested review of D35904: textproc/linux-c7-expat-devel: Add new port.
Mon, Jul 25, 9:04 AM
zirias requested review of D35903: security/linux-c7-openssl-devel: Add new port.
Mon, Jul 25, 9:04 AM
zirias requested review of D35902: devel/linux-c7-zlib-devel: Add new port.
Mon, Jul 25, 9:03 AM

Sun, Jul 24

zirias updated the diff for D35889: Mk/Uses/compiler.mk: Support newer C++ standards.

Extend VALID_ARGS

Sun, Jul 24, 4:41 PM
zirias updated the diff for D35889: Mk/Uses/compiler.mk: Support newer C++ standards.

Fix min version for llvm

Sun, Jul 24, 1:01 PM
zirias added a comment to D35889: Mk/Uses/compiler.mk: Support newer C++ standards.

Remark: I'm not too sure about the support for newer C++ standards as this would probably also require support in STL/libstdc++...

Sun, Jul 24, 12:47 PM
zirias requested review of D35889: Mk/Uses/compiler.mk: Support newer C++ standards.
Sun, Jul 24, 12:45 PM

Tue, Jul 19

zirias added a comment to D35805: net/linphone: Update to 4.4.8.

@hselasky I will do so ASAP (read: as soon as my builder machine is done rebuilding my package repos).

Tue, Jul 19, 8:56 PM
zirias added a comment to D35805: net/linphone: Update to 4.4.8.

Will you handle this one-liner or should I?

Tue, Jul 19, 1:50 PM
zirias added a comment to D35805: net/linphone: Update to 4.4.8.

I'm not sure exactly what is going on, but this patch fixes it on my side:
[...]

And, looking at upstream sources again, this patch looks perfectly sane. We don't need this "build" subdir, it seems its only purpose is building some packages with bundled deps.

Tue, Jul 19, 11:34 AM
zirias added a comment to D35805: net/linphone: Update to 4.4.8.

I can't reproduce that.

Tue, Jul 19, 10:41 AM
zirias added a comment to D35805: net/linphone: Update to 4.4.8.

FYI - FreeBSD 14-current
[...]

Yes, something is fishy with the smake magic in bctoolbox, but it doesn't seem to have any effect. It builds one binary (linphone) and one shared lib (libapp-plugin.so) as it should.

Tue, Jul 19, 9:49 AM

Fri, Jul 15

zirias updated the summary of D35805: net/linphone: Update to 4.4.8.
Fri, Jul 15, 9:06 AM
zirias updated the diff for D35805: net/linphone: Update to 4.4.8.

Add explanation about RPATH, as it's non-obvious

Fri, Jul 15, 7:26 AM

Thu, Jul 14

zirias updated the diff for D35805: net/linphone: Update to 4.4.8.

Avoid rebuilding during stage phase.

Thu, Jul 14, 5:01 PM

Wed, Jul 13

zirias updated the diff for D35804: net/liblinphone: Update to 5.1.32.

Add missing include <ctime> in auto-generated C++ headers

Wed, Jul 13, 9:19 PM
bofh awarded D35804: net/liblinphone: Update to 5.1.32 a Like token.
Wed, Jul 13, 4:23 PM
zirias requested review of D35805: net/linphone: Update to 4.4.8.
Wed, Jul 13, 1:44 PM
zirias requested review of D35804: net/liblinphone: Update to 5.1.32.
Wed, Jul 13, 1:44 PM

Tue, Jul 12

zirias added a comment to D35782: bsd.sites.mk: Add GH_CHERRYPICK variable.
In D35782#812003, @mat wrote:

The way to go is to put the patches in the files directory in the port, and git add them.

Tue, Jul 12, 12:53 PM
zirias added a comment to D35782: bsd.sites.mk: Add GH_CHERRYPICK variable.

Sure, but isn't "same repo" the common case? I'd argue it simplifies that case, you get rid of hand-crafted PATCH_SITES, PATCH_FILES and PATCH_DIST_STRIP (or extra arg for the files)... and if GitHub ever changed how to access patches for individual commits, it could be handled in a central place.

Tue, Jul 12, 12:08 PM
zirias added a comment to D35782: bsd.sites.mk: Add GH_CHERRYPICK variable.

Rationale: it happened to me a few times that I had to patch upstream source for FreeBSD, and my patch was later accepted and merged. For github projects, this would add a simple way to apply the patch from upstream as long as it's still missing from the latest release.

Tue, Jul 12, 11:18 AM
zirias requested review of D35782: bsd.sites.mk: Add GH_CHERRYPICK variable.
Tue, Jul 12, 11:13 AM

Mon, Jul 11

zirias added a comment to D35195: comms/opencbm-kmod: Remove unused DRIVER_MODULE devclass on recent main..

Patch just landed upstream: https://github.com/OpenCBM/OpenCBM/commit/6e438c1b32d39e5238f65246e192c89b22fdfd82 – sorry for the long delay.

Mon, Jul 11, 7:04 PM
zirias abandoned D34322: pam_unix: Use suid root helper to authenticate.

Abandoned in favor of new port security/unix-selfauth-helper that can be used with pam_exec by screen lockers once https://reviews.freebsd.org/D35169 is fixed on release branches.

Mon, Jul 11, 2:51 PM
zirias abandoned D35728: lang/python39: Fix build with LibreSSL 3.5.
Mon, Jul 11, 2:44 PM

Jul 8 2022

zirias requested review of D35757: multimedia/makemkv: Add support for aarch64.
Jul 8 2022, 1:57 PM

Jul 6 2022

zirias requested review of D35729: net/qt5-network: Runtime fixes with libressl.
Jul 6 2022, 11:21 AM
zirias requested review of D35728: lang/python39: Fix build with LibreSSL 3.5.
Jul 6 2022, 11:21 AM

Jun 27 2022

zirias updated the diff for D35348: x11/xscreensaver: Update to 6.04.

Re-apply accidentally reverted icon fix.

Jun 27 2022, 10:15 AM
zirias updated the diff for D35348: x11/xscreensaver: Update to 6.04.

Add run dep on p5-LWP-Protocol-https

Jun 27 2022, 10:10 AM

Jun 19 2022

zirias updated the diff for D35347: security/unix-selfauth-helper: Add new port.

Fix building as non-root.

Jun 19 2022, 10:30 AM

Jun 13 2022

zirias updated the diff for D35348: x11/xscreensaver: Update to 6.04.

Fix icon in xscreensaver-start.desktop

Jun 13 2022, 10:15 AM
zirias updated the diff for D35348: x11/xscreensaver: Update to 6.04.

Latest upstream version is now 6.04

Jun 13 2022, 10:04 AM
zirias retitled D35348: x11/xscreensaver: Update to 6.04 from x11/xscreensaver: Update to 6.03 to x11/xscreensaver: Update to 6.04.
Jun 13 2022, 10:03 AM

Jun 8 2022

zirias updated the diff for D35347: security/unix-selfauth-helper: Add new port.

Remove example file, improve manpage instead.

Jun 8 2022, 11:27 AM

Jun 5 2022

zirias updated the diff for D35347: security/unix-selfauth-helper: Add new port.

Add example pam service policy configuration

Jun 5 2022, 2:03 PM

May 30 2022

zirias updated the diff for D35348: x11/xscreensaver: Update to 6.04.

rebase on latest main

May 30 2022, 10:44 AM
zirias updated the diff for D35348: x11/xscreensaver: Update to 6.04.

don't hardcode /usr/local in pam configuration

May 30 2022, 9:55 AM
zirias added a comment to D35347: security/unix-selfauth-helper: Add new port.

This depends on the fix in https://reviews.freebsd.org/D35169

May 30 2022, 8:45 AM
zirias requested review of D35348: x11/xscreensaver: Update to 6.04.
May 30 2022, 8:43 AM
zirias requested review of D35347: security/unix-selfauth-helper: Add new port.
May 30 2022, 8:43 AM

May 22 2022

zirias added a comment to D35169: pam_exec: fix segfault when existing auth token is null on `pam_sm_set_cred`.

Would someone mind to commit this, please? It enables correct operation of a suid-root helper that will probably be necessary to support newer versions of xscreensaver.
I also wonder whether this might warrant an EN for the next round of patches?

May 22 2022, 10:35 AM

May 21 2022

zirias added a comment to D35195: comms/opencbm-kmod: Remove unused DRIVER_MODULE devclass on recent main..
In D35195#799397, @jhb wrote:

Do you have a relationship with upstream to post the patch there?

May 21 2022, 7:12 AM

May 17 2022

zirias added a comment to D34322: pam_unix: Use suid root helper to authenticate.

Ok, with the patch from https://reviews.freebsd.org/D35169 applied, my new helper seems to work very well, I couldn't find any problems myself any more (added another commit with a few minor improvements).

May 17 2022, 12:09 PM

May 16 2022

zirias added a comment to D35169: pam_exec: fix segfault when existing auth token is null on `pam_sm_set_cred`.

So, does this patch look fine?

May 16 2022, 12:10 PM
zirias added a comment to D35169: pam_exec: fix segfault when existing auth token is null on `pam_sm_set_cred`.

Is it ever expected there *is* an authtok available when setcred is called? If not, wouldn't this still fail (and just avoid the crash)?

May 16 2022, 10:26 AM

May 13 2022

zirias accepted D35195: comms/opencbm-kmod: Remove unused DRIVER_MODULE devclass on recent main..

Looks fine for now, trusting your word the version constant is correct, hehe. Should probably try to take this upstream (and then update the port) ;)

May 13 2022, 5:57 PM

May 9 2022

zirias added a comment to D34322: pam_unix: Use suid root helper to authenticate.

Ok, here's some first code that seems to work for me, would be happy to get some input on it: https://github.com/Zirias/unix-selfauth-helper

May 9 2022, 11:04 AM

May 4 2022

zirias added a comment to D34322: pam_unix: Use suid root helper to authenticate.
In D34322#796042, @des wrote:

The correct solution is to replace the whole NSS + PAM mess with a unified identity and authentication service

I have to agree *this* is the correct and clean solution indeed. I won't volunteer for it either, cause I think I'm lacking necessary experience for writing such a service.

May 4 2022, 1:09 PM
zirias added a comment to D34322: pam_unix: Use suid root helper to authenticate.
In D34322#796040, @des wrote:

That's what I've been saying all along, so what are we arguing about?

May 4 2022, 1:02 PM
zirias added a comment to D34322: pam_unix: Use suid root helper to authenticate.
In D34322#795906, @des wrote:

You don't need an alternate pam_unix(8). As I've already told Felix, you can invoke a hypothetical setuid helper using pam_exec(8).

Sure this can be used, but how is it less of a hack? I'd still say the fact that the caller of PAM functions *sometimes* needs root privileges, depending on which modules are configured, is, uhm, conceptually wrong.

May 4 2022, 8:32 AM

May 2 2022

zirias added a comment to D34322: pam_unix: Use suid root helper to authenticate.

I'd also argue such an external helper is the much worse hack:

May 2 2022, 11:30 PM