@pmatilai commented on this pull request.


> @@ -0,0 +1,17 @@
+FROM registry.fedoraproject.org/fedora:41
+
+RUN dnf install -y \
+    gcc \
+    gcc-c++ \

Oh I managed to gloss over that somehow, looking for something else I guess.
I would really contain this to containers only. It's not a lot of code but the 
less options there are the less options there are to maintain and worry about, 
and downloading and installing stuff from the net from our makefiles is not 
something I'm terribly comfortable with. 

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3666#discussion_r2005242117
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3666/review/2702006...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
https://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to