From nobody Tue Oct 10 13:50:59 2023 X-Original-To: ports-bugs@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4S4ckC4S7gz4wnRQ for ; Tue, 10 Oct 2023 13:50:59 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4S4ckC1crjz4LkX for ; Tue, 10 Oct 2023 13:50:59 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1696945859; a=rsa-sha256; cv=none; b=o59dAi8Gbu0CJ7xSN036uelA/zdX/Hrjcv5V+VhJmXIWTJ8cUnQkpknM2TiZlya8HTlulg Cc016esbsNqGDyl2wlwiuRBkd20++ujkdy52djzLT8/e6lhKLQEdo/ppGZcCodGU/2OEBI ayGkor3kpz1XuPQQa/LzUyq8gCx8A6prB6PVo1q+qN4htq/DnX06i/ppMQv1bg3EdMFf/a zdsigCtBMgNsYLMwmlkzoCSXxSBwZPtmSGCWGKszyiusMkKHJm8fn+qHKzFMrRIM7+LE9O qRtAjxCanUSaL7gyDhlzwnYd1WD4NUmxMotYEvBQhZOsrsbQkOo9Xu0KygGL0w== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1696945859; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=9itFv7oakcf644FeTgpj1oXdTpQvnlDYa5oPHHdd1Fs=; b=ikPwuOmoQc2wybfqKm/k6x3S8eC0TdRF3mqwQ/+oME2KXuy4t2QYKsS5dkNrRywNyKAY3k zIfjD+ihZ3V556kqlU9/R7WBdXfuQlmVj/tLC9onPJa4UrGVop/+7pu3OnrjDtdWEnqoSF Uu03Nz/vg0EWdc0WwBwem02FLCmxKUmu0aHTP7ESPAoS9wb1Jp/JprBMtmqSUVKCHCx5qG 05hIiuXqoxdgII4MbO3Y95nY7xHthaUucw64yWIvqDrls5w+TcgfaEHk15Hz/C6fYFMsTL XvjETMvkSuPwn/8L+fwKhX9ZGgXwv3RhbwHroV/vGQRGe5N9vAuzZlu/lr3eUw== Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2610:1c1:1:606c::50:1d]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4S4ckC0hXszghH for ; Tue, 10 Oct 2023 13:50:59 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from kenobi.freebsd.org ([127.0.1.5]) by kenobi.freebsd.org (8.15.2/8.15.2) with ESMTP id 39ADox3e027263 for ; Tue, 10 Oct 2023 13:50:59 GMT (envelope-from bugzilla-noreply@freebsd.org) Received: (from www@localhost) by kenobi.freebsd.org (8.15.2/8.15.2/Submit) id 39ADoxLa027262 for ports-bugs@FreeBSD.org; Tue, 10 Oct 2023 13:50:59 GMT (envelope-from bugzilla-noreply@freebsd.org) X-Authentication-Warning: kenobi.freebsd.org: www set sender to bugzilla-noreply@freebsd.org using -f From: bugzilla-noreply@freebsd.org To: ports-bugs@FreeBSD.org Subject: [Bug 262032] [NEW PORT] devel/xnvme: Cross-platform libraries and tools for NVMe devices Date: Tue, 10 Oct 2023 13:50:59 +0000 X-Bugzilla-Reason: AssignedTo X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Ports & Packages X-Bugzilla-Component: Individual Port(s) X-Bugzilla-Version: Latest X-Bugzilla-Keywords: feature X-Bugzilla-Severity: Affects Only Me X-Bugzilla-Who: fuz@FreeBSD.org X-Bugzilla-Status: Open X-Bugzilla-Resolution: X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: ports-bugs@FreeBSD.org X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated List-Id: Ports bug reports List-Archive: https://lists.freebsd.org/archives/freebsd-ports-bugs List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-freebsd-ports-bugs@freebsd.org X-BeenThere: freebsd-ports-bugs@freebsd.org MIME-Version: 1.0 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D262032 --- Comment #10 from Robert Clausecker --- (In reply to Karl Bonde Torp from comment #9) Hi Karl, > I think the way to go for the architectures is using an "ONLY_FOR_ARCHS= =3D amd64", > since this is the only thing we are currently testing - which is also why= we > haven't found the issues you're seeing. Please don't do that. If people were to generally only enable packages for architectures they personally tested, rare architectures like riscv64 would receive barely any packages at all. Only set ONLY_FOR_ARCHS if you know th= at the package will only work for the listed architectures. That you didn't t= est for others doesn't mean it won't work on these. You can always mark indivi= dual architectures as broken when problems are discovered, but unless known not = to be the case, you should presume that your code works on any architecture. > For both the Manpages and bash completions I will have to do some work u= pstream to improve the flexibility of our `meson.build` file. Sounds great. We can temporarily commit the port without this flexibility = and later update it with new options if you like. > In the latest patch I did put "OPTIONS_SUB=3D yes". The section =C2=A7 5.= 14.3.1 Porter's Handbook is not very detailed, but it seems that I have don= e everything that is needed? That's half the game. This option causes placeholders like %%TESTS%% to be defined. You need to tag each file in pkg-plist with what option causes it= to be installed by placing the placeholder right in front. Only files for whi= ch the appropriate option is on are then shipped. E.g. do %%TESTS%%bin/xnvme_tests_buf to only install this file in case the TESTS option is set. (For DOCS, the placeholder is %%PORTDOCS%% for historical reasons). It is okay if your bu= ild scripts install files that you don't want to keep, as long as they are list= ed for some (possibly currently disabled) option in pkg-plist. Such files won= 't be included in the package. However, it's of course better if you try to a= void doing so. > I believe that installing both CLI tools and libraries is a sane default,= and I don't believe there is any reason to split them up into two ports at= the moment. Okay, then let's go with that. > As you point out, this is available online - and I don't think there is a= need to include this in the port. Sure, but please be mindful of users who lack internet access for some reas= on.=20 Your tools look like they will be particularly useful for systems recovery = and maintenance, which are typical scenarios where this might be the case. > I will submit a new patch at that point, thanks for all your help! Looking forwards to that and thanks for all the work. --=20 You are receiving this mail because: You are the assignee for the bug.=