Please consider removing the ONLY_FOR_ARCHS line when updating this port. I have managed to successfully compile the port for armv7 at least.

In D33162#750038, @contact_artur-rojek.eu wrote:

How about extract_s32/extract_s64?

I would prefer shorter names. signed_extend32 is really unwieldy. Also, the name does not really explain that the operation is really about extracting from bitfields.

As discussed on IRC, I propose that such a helper should have three arguments:

LGTM. Once this is committed, I'll go ahead and report our patches up stream.

Note that there's no OpenMP with clang on armv7. It's a bit weird to take this out because now way more ports will fail to build on armv7. Would have been better to make compiler:openmp only chose gcc on architectures where LLVM doesn't implement OpenMP.

In D32220#726921, @yasu wrote:

In D32220#726911, @diizzy wrote:

I would also recommend you to pursue submitter to submit patches upstream and/or reference backports.

It seems changes in both patch-Config_exports.h and patch-Config_exports.l are already exist in upstream repository.
But it not clear if submitter created them by himself or took them from upstream repository.

As already discussed, I have not asked for a STATIC option to be added not have I asked for the test suite to be disabled. In fact, I do not wish to maintain these changes. Please remove them as already discussed on IRC.

Commit message could use an explicit reference to dummy message catalogues being added and a link to the announce file (changelog).

Looks good to me.

@lbartoletti USE_GITHUB does not seem possible: this is an autotools project and the release tarballs have pre-generated autotools files that aren't in the repository. I've previously been told off for using USE_GITHUB and regenerating the configure script in such a case.

After private discussion with @tcberner, how about

I have intentionally chosen this COMMENT to highlight that this is not an ordinary backup program, but rather performs some RAID-like functionality.

Patch successfully on arm64 13.0 RELEASE. Applying it makes armv7 Go binaries work.

devel/schilybase: refactor bootstrap smake logic

Rebase onto current main branch as requested by otis.

Address all feedback by otis, implementing the changes he suggested.

• MOVED: fix wrong port name in sysutils/sformat entry
• devel/schilybase: unsilence post-extract commands

Will fix the MOVED issue shortly. Please let me know if you still want the silencing to be taken out, given that it seems to be recommended by the Porter's handbook and is general practice in many other ports.

sysutils/schilyutils: rename bin/compare to bin/scompare

Add forgotten USE_LDCONFIG

Mark items addressed by Diff 90898 as done.

address code review feedback by mat

archivers/star: simplify Makefile

sysutils/sformat: obsolete; subsumed under sysutils/schilyutils

archivers/star: do not install an mt binary