From nobody Mon Nov 06 19:55:14 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 4SPMX30vCnz4ytfv for ; Mon, 6 Nov 2023 19:55:15 +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 4SPMX261LJz3L1k for ; Mon, 6 Nov 2023 19:55:14 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1699300514; 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=SnAVnFIjMzgrf6tyq46ZQGUcpNJ0yCX+TkUQ9hHH0pY=; b=LAb1JhFmMUxe/JAKgNAp5hN9PDz+EwGE1yz4gNaQYEZle4Gj7eQAT9VjtfygIfbl8jwg+A uK5t9G41n3fgEiU3wO1EJXTiypEyeQ12Jaa6AUlrfxcDRFOFS6XyZuTjBOLM4GwA4yIsk6 wsQvos1obdvzT70Co420oc/chXGecoZ/EqtMH1adRVYeRpj9Lh79ZBeZblj77yL+hn9RFt UDHDqZCFG41zue8sMIrRBe9qUW7/W4K/aqVZ54wgCS9dgych2h84NwDTK+E5rQSOmgkmOI XZJ0ouvaCo81dpN8XYi1dJKW8QyiZL6k50aa61/DesPFDKYX/S551K4CsQ3GXA== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1699300514; a=rsa-sha256; cv=none; b=mLHPgVZCgE4g8iV+VBVMLhieNR9w8e9v7v+ph/noJ98DPPHjvfofbmL6gutaV3oMnTtjS8 KV/VnhiukJyMpNdLUw1yLbtEi3Wu9zn6YSNuiXh2XDyM0S+YXHXoW+JgNtJIGE5gbCAW1f Urbb4mQ0uyZZi7nJt7oKaWYOO/WqL68aeALwplTI+mhxhcpKi79sjCFlXVj7mfaIHGGLCj kXKTLUcazlyCX3bJxM4uF2RNyTJYt2lT8QBRck09ivPfHrtNh0Y5cHHYX/05nyM/P1nzku /tQN4f8PfJANFu//aSR2WC49b56O4sb3XcCF8SzSyXc3RuCtt1GMf34Xwtu87g== 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 4SPMX24j9dz5Br for ; Mon, 6 Nov 2023 19:55:14 +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 3A6JtEko063422 for ; Mon, 6 Nov 2023 19:55:14 GMT (envelope-from bugzilla-noreply@freebsd.org) Received: (from www@localhost) by kenobi.freebsd.org (8.15.2/8.15.2/Submit) id 3A6JtEj1063421 for ports-bugs@FreeBSD.org; Mon, 6 Nov 2023 19:55:14 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 262318] [NEW PORT] lang/freebasic Date: Mon, 06 Nov 2023 19:55:14 +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: X-Bugzilla-Severity: Affects Some People 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=3D262318 --- Comment #3 from Robert Clausecker --- Thank you for revisiting this. I hope you didn't loose to much data. The port still looks a bit weird in some ways: - you supply what you say to be an info page, but it looks like it's a man page. Please treat it as such. - again, do not hard code gmake. Use ${MAKE} instead. It will resolve to= the correct make command as you have added USES=3Dgmake. - note that ARCH is armv6 or armv7 for arm, but your code looks like it expects it to be arm. Check if you can get that fixed. Same for what you call = x86. On FreeBSD, it's called i386. - the do-build very weird. I'll see if I can get them simplified. - instead of REINPLACE_CMD in do-install, it might be better to use the .in mechanism (cf. =C2=A7 9.5 Porter's Handbook) to patch in the correct PRE= FIX, ARCH and DISTVERSION. - do not hard-code /usr/local, use PREFIX and LOCALBASE instead (as appropriate). Your port must not assume anything is found in /usr/local= and must not hard code this path anywhere. Instead, rely on the variables passed in by the ports framework. You may want to add USES=3Dlocalbase, which = adds -L${LOCALBASE}/lib and -I${LOCALBASE}/include to the appropriate variabl= es. - in inc/crt/errno.bi, why not just #include to get the list of errno values? This is much more future-proof than copying the list of values. - The note you add in lang/freebasic/files/patch-inc_crt_freebsd_ctype.bi contradicts the LGPL license the port is supposed to have. Also, by add= ing this file to the port, you have already distributed it apart from the FreeBASIC distribution. Please remove the note or bring it into complia= nce with the license. Same for the other notes. - Please remove -Wfatal-errors from the Makefile. Future compiler versions may have different error messages and we don't want the port to stop building because the compiler feels like warning about some random irrelevant thi= ng.=20 Same goes for -Werror and similar. - Do not override the CFLAGS you receive from the environment, only ever append to them. Ports must obey CFLAGS given to them by the user or the ports framework. Same for CPPFLAGS, LDFLAGS, etc. - Do not set CC in your port. Instead use whatever value CC has been set = to by the ports framework. If you need gcc to build this port, add USE_GCC=3D= yes to the port makefile, which will cause it to select a version of gcc. Do n= ot try to guess what the compiler is. - Do not set prefix. Obey whatever value PREFIX has been set to by the po= rt framework. This value may or may not be /usr/local. - if you need commands like dos2unix installed, make sure to add them to BUILD_DEPENDS. All dependencies of the port must be listed there. I recommend you use Poudriere to build-test the port. Poudriere builds the port in a jail that only has the listed dependencies installed, making it easy to check if you forgot any. - instead of running dos2unix in your makefile, you can add USES=3Ddos2uni= x to the port makefile, which will automatically convert all line endings in = the project. See =C2=A7 17.19 Porter's Handbook for details. - to use ncurses, add USES=3Dncurses to the port makefile. Your port MUST= obey NCURSESINC, NCURSESLIB, NCURSESRPATH, etc. (see Mk/Uses/ncurses.mk) to f= ind the correct ncurses. There can be multiple copies of the library instal= led and only the one these variables indicate is correct. - please remove WWW: from pkg-descr and instead add a WWW variable to the = port Makefile. This is another ecent change in the port framework. --=20 You are receiving this mail because: You are the assignee for the bug.=