devel/sd-mux-ctrl: Rename to match upstream
ClosedPublic
Actions

Authored by sbz on Apr 12 2021, 5:30 PM.

Details

Reviewers

swills

Commits

R11:ae9ec3576cee: devel/sd-mux-ctrl: Rename to match upstream

Summary

Rename devel/sd-mux to devel/sd-mux-ctrl and remove dupplicate port
sysutils/sd-mux-ctrl added recently. As the port names were different
addport -y didn't catch the dupplicate when looking at the catalog tree
INDEX. Also add entries into MOVED.

Approved by: swills

Test Plan

Ok poudriere testport -o devel/sd-mux-ctrl
Ok port test

Diff Detail

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 38562
Build 35451: arc lint + arc unit

Event Timeline

sbz created this revision.Apr 12 2021, 5:30 PM

Herald added a subscriber: mat. · View Herald TranscriptApr 12 2021, 5:30 PM

sbz requested review of this revision.Apr 12 2021, 5:30 PM

Harbormaster completed remote builds in B38538: Diff 87328.Apr 12 2021, 5:30 PM

license

Harbormaster completed remote builds in B38539: Diff 87329.Apr 12 2021, 5:36 PM

swills added inline comments.Apr 13 2021, 12:00 PM

devel/sd-mux-ctrl/files/patch-src_CMakeLists.txt
12	Someone who knows CMake better should say if this patch or the other are better in some way or more upstream-able. I suspect the other one is. Have we tried to get either upstream? I don't think I bothered with mine, but we should. Upstream-ing shouldn't hold up this change, but we should decide which patch will work upstream and stick with that for this change.
devel/sd-mux-ctrl/pkg-descr
4–5	Might be nice to preserve the other link to the SDWire wiki page so folks can figure out what this is better. It's OK to have two links in pkg-descr, btw.

preserve the link to the wiki page in pkg-descr

Harbormaster completed remote builds in B38562: Diff 87386.Apr 13 2021, 4:44 PM

sbz marked an inline comment as done.Apr 13 2021, 4:45 PM

sbz added inline comments.Apr 13 2021, 4:48 PM

devel/sd-mux-ctrl/files/patch-src_CMakeLists.txt
12	I just kept the patch as before and it works as expected, i.e. nothing was changed. For the upstream question of the patches, I'm agreed and I think it should be chased during the next release update.

I just kept the patch as before and it works as expected, i.e. nothing was changed. For the upstream question of the patches, I'm agreed and I think it should be chased during the next release update.

Ok, fine, I'll do it.

devel/sd-mux-ctrl/files/patch-src_CMakeLists.txt
12	I just kept the patch as before and it works as expected, i.e. nothing was changed. For the upstream question of the patches, I'm agreed and I think it should be chased during the next release update.

This revision is now accepted and ready to land.Apr 13 2021, 6:03 PM

emaste added subscribers: gonzo, emaste.Apr 13 2021, 6:53 PM

emaste added inline comments.

devel/sd-mux-ctrl/files/patch-src_CMakeLists.txt
12	I think the other one is more upstreamable and we should use that one. That said, @gonzo tried and they were unresponsive. There's a link to the upstream submission somewhere in the thread at https://twitter.com/kernelnomicon/status/1284230380887789568 I think.