CVS commit: src/distrib/atari/floppies/install
Module Name:src Committed By: rin Date: Wed May 11 10:47:10 UTC 2022 Modified Files: src/distrib/atari/floppies/install: Makefile Log Message: Drop IEEE 802.11 support from ifconfig(8). Reduce ~9KB, and atari builds again for me. To generate a diff of this commit: cvs rdiff -u -r1.7 -r1.8 src/distrib/atari/floppies/install/Makefile Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
CVS commit: src/distrib/atari/floppies/install
Module Name:src Committed By: rin Date: Wed May 11 10:47:10 UTC 2022 Modified Files: src/distrib/atari/floppies/install: Makefile Log Message: Drop IEEE 802.11 support from ifconfig(8). Reduce ~9KB, and atari builds again for me. To generate a diff of this commit: cvs rdiff -u -r1.7 -r1.8 src/distrib/atari/floppies/install/Makefile Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files. Modified files: Index: src/distrib/atari/floppies/install/Makefile diff -u src/distrib/atari/floppies/install/Makefile:1.7 src/distrib/atari/floppies/install/Makefile:1.8 --- src/distrib/atari/floppies/install/Makefile:1.7 Sun Jul 28 10:28:49 2019 +++ src/distrib/atari/floppies/install/Makefile Wed May 11 10:47:10 2022 @@ -1,4 +1,4 @@ -# $NetBSD: Makefile,v 1.7 2019/07/28 10:28:49 martin Exp $ +# $NetBSD: Makefile,v 1.8 2022/05/11 10:47:10 rin Exp $ IMAGE= sysinst.fs IMAGESIZE?= 1440k @@ -7,4 +7,5 @@ USE_SYSINST= yes IMGMAKEFSOPTIONS= -o bsize=4096,fsize=512,density=4096 HACK_CURSES=yes +NOIEEE80211=1 .include "../common/Makefile.images"
Re: CVS commit: src/distrib/atari/floppies/install
(note I don't even know how long atari build has been broken) This problem often rises when build failure is caused by MI changes without particular annoucements. If someone[tm] can update our autobuild script, it would be great.. - less frequent (weekly or semi-monthly?) builds for tierII ports - proper notification to MD port list on autobuild failure, as i386 does to current-users --- Izumi Tsutsui
Re: CVS commit: src/distrib/atari/floppies/install
Joerg Sonnenberger writes: > On Sun, Apr 26, 2015 at 10:07:32AM -0400, Greg Troxel wrote: >> A broken build is evidence that the prior commits were not tested, >> and thus need fixing. Objecting to fixing the build without testing >> should be a far lower priority than objecting to commits that break >> the build. > The problem here is that the original commit that triggered the > overflow can and often enough is perfectly reasonable on all platforms > without artifically low size constraints. What is needed to get at > least basic regression testing done in an emulator? Fair enough and a good point about emulator testing being really helpful. pgpGjBHUeCXLj.pgp Description: PGP signature
Re: CVS commit: src/distrib/atari/floppies/install
On Sun, Apr 26, 2015 at 10:07:32AM -0400, Greg Troxel wrote: > > Izumi Tsutsui writes: > > > joerg@ wrote: > > > >> > "Please don't commit untested and broken fix." > >> > "Please file a PR instead if you can't test it." > >> > >> I agree with Christos that leaving the build broken is worse. The > >> alternative band aids are much more involved, too. That shouldn't stop > >> the second point from that list. > > > > If you claim leaving the build broken is worse than commiting untested code, > > you should ask to update our commit guideline first: > > > > http://www.netbsd.org/developers/commit-guidelines.html > > Perhaps we should. The problems are: > > A broken build is evidence that the prior commits were not tested, and > thus need fixing. Objecting to fixing the build without testing > should be a far lower priority than objecting to commits that break > the build. The problem here is that the original commit that triggered the overflow can and often enough is perfectly reasonable on all platforms without artifically low size constraints. What is needed to get at least basic regression testing done in an emulator? Joerg
Re: CVS commit: src/distrib/atari/floppies/install
joerg@ wrote: > > > > "Please don't commit untested and broken fix." > > > > "Please file a PR instead if you can't test it." > > > > > > I agree with Christos that leaving the build broken is worse. The > > > alternative band aids are much more involved, too. That shouldn't stop > > > the second point from that list. > > > > If you claim leaving the build broken is worse than commiting untested code, > > you should ask to update our commit guideline first: > > Bumping the size of a floppy is definitely the smaller evil. Can you > please try to be reasonable? I'm not sure what's your definition of "definitely", but bumping the size of a floppy is fatal on atari, as you can see in the first message of this thread. Our commit guideline is quite reasonable: >> ask a developer who is familiar with that part of the system for >> review. --- Izumi Tsutsui
Re: CVS commit: src/distrib/atari/floppies/install
Izumi Tsutsui writes: > joerg@ wrote: > >> > "Please don't commit untested and broken fix." >> > "Please file a PR instead if you can't test it." >> >> I agree with Christos that leaving the build broken is worse. The >> alternative band aids are much more involved, too. That shouldn't stop >> the second point from that list. > > If you claim leaving the build broken is worse than commiting untested code, > you should ask to update our commit guideline first: > > http://www.netbsd.org/developers/commit-guidelines.html Perhaps we should. The problems are: A broken build is evidence that the prior commits were not tested, and thus need fixing. Objecting to fixing the build without testing should be a far lower priority than objecting to commits that break the build. A broken build, or a build that succeeds but fails to work is a serious problem because it prevents bisecting to find bugs. I've seen this in current/i386 pretty often, less so recently. During a ~week that the build was broken many commits happened, and some of those were trouble, and it was a mess to sort out. So arguably no commits should be allowed at all during a time when the build is broken or there are sudden significant new test failures, other than fixing the build. pgpPVycMH2F2d.pgp Description: PGP signature
Re: CVS commit: src/distrib/atari/floppies/install
On Sun, Apr 26, 2015 at 10:39:00PM +0900, Izumi Tsutsui wrote: > Please don't assume all tier II users check auto-build reports > even every month. I would expect portmaster at least to check much more often than once a month or at least request notification. Joerg
Re: CVS commit: src/distrib/atari/floppies/install
On Sun, Apr 26, 2015 at 10:54:41PM +0900, Izumi Tsutsui wrote: > joerg@ wrote: > > > > "Please don't commit untested and broken fix." > > > "Please file a PR instead if you can't test it." > > > > I agree with Christos that leaving the build broken is worse. The > > alternative band aids are much more involved, too. That shouldn't stop > > the second point from that list. > > If you claim leaving the build broken is worse than commiting untested code, > you should ask to update our commit guideline first: Bumping the size of a floppy is definitely the smaller evil. Can you please try to be reasonable? Joerg
Re: CVS commit: src/distrib/atari/floppies/install
joerg@ wrote: > > "Please don't commit untested and broken fix." > > "Please file a PR instead if you can't test it." > > I agree with Christos that leaving the build broken is worse. The > alternative band aids are much more involved, too. That shouldn't stop > the second point from that list. If you claim leaving the build broken is worse than commiting untested code, you should ask to update our commit guideline first: http://www.netbsd.org/developers/commit-guidelines.html >> 1. Commit only code you are familiar with. >> If you are not sure if the code you plan to commit is acceptable >> (e. g. when taking code that was submitted with a problem report), >> ask a developer who is familiar with that part of the system for >> review. : >> 5. Commit only code that you have tested. >> Make sure that your code actually works as expected, by compiling >> and running the code that is affected by your change with your >> system's tools. --- Izumi Tsutsui
Re: CVS commit: src/distrib/atari/floppies/install
On Apr 26, 10:39pm, tsut...@ceres.dti.ne.jp (Izumi Tsutsui) wrote: -- Subject: Re: CVS commit: src/distrib/atari/floppies/install | christos@ wrote: | | > | "Please don't commit untested and broken fix." | > | "Please file a PR instead if you can't test it." | > | > I can't test, and it is broken already. For months... | > Having the build broken is bad, because it means that any changes | > after the break are not even compile-tested. | | Please file a PR if you notice any problems. | | I have tested it netbsd-7, but it's unlikey to test -current periodically | before release. | | > Again, I was just saying that we should consider moving ports | > that people don't seem care about to tier III. | | Please file a PR for poor tier II users. | Please don't assume all tier II users check auto-build reports | even every month. I will do that :-) It is easier than fixing the problem... christos
Re: CVS commit: src/distrib/atari/floppies/install
christos@ wrote: > | "Please don't commit untested and broken fix." > | "Please file a PR instead if you can't test it." > > I can't test, and it is broken already. For months... > Having the build broken is bad, because it means that any changes > after the break are not even compile-tested. Please file a PR if you notice any problems. I have tested it netbsd-7, but it's unlikey to test -current periodically before release. > Again, I was just saying that we should consider moving ports > that people don't seem care about to tier III. Please file a PR for poor tier II users. Please don't assume all tier II users check auto-build reports even every month. --- Izumi Tsutsui
Re: CVS commit: src/distrib/atari/floppies/install
On Apr 26, 11:52am, tsut...@ceres.dti.ne.jp (Izumi Tsutsui) wrote: -- Subject: Re: CVS commit: src/distrib/atari/floppies/install | You always miss the point. You take everything so seriously; it is not the end of the world: it did not work before the change and it does not work now (well it probably works now, I don't know). | "Please don't commit untested and broken fix." | "Please file a PR instead if you can't test it." I can't test, and it is broken already. For months... Having the build broken is bad, because it means that any changes after the break are not even compile-tested. | What I wrote was: | | >> I agree you can blame port masters if they leave their ports broken | >> more than *weeks*. | : | >> If you claim port-masters must check buildable state *everyday* | >> against all MI changes without review or announcement, I'll resign | >> from all maintainership. | | Claiming "tier III since nobody seems to care" without PR is quite unfair. Again, I was just saying that we should consider moving ports that people don't seem care about to tier III. christos
Re: CVS commit: src/distrib/atari/floppies/install
On Sun, Apr 26, 2015 at 11:52:08AM +0900, Izumi Tsutsui wrote: > christos@ wrote: > > > | I just think "buildability fix" won't help users, > > | as we discussed in the past. > > | http://mail-index.netbsd.org/source-changes-d/2012/08/01/msg005113.html > > > > Well, it does not help being broken for months... I say tier III, since > > nobody seems to care. Anyway, fixed for now. > > You always miss the point. > > "Please don't commit untested and broken fix." > "Please file a PR instead if you can't test it." I agree with Christos that leaving the build broken is worse. The alternative band aids are much more involved, too. That shouldn't stop the second point from that list. Joerg
Re: CVS commit: src/distrib/atari/floppies/install
christos@ wrote: > | I just think "buildability fix" won't help users, > | as we discussed in the past. > | http://mail-index.netbsd.org/source-changes-d/2012/08/01/msg005113.html > > Well, it does not help being broken for months... I say tier III, since > nobody seems to care. Anyway, fixed for now. You always miss the point. "Please don't commit untested and broken fix." "Please file a PR instead if you can't test it." What I wrote was: >> I agree you can blame port masters if they leave their ports broken >> more than *weeks*. : >> If you claim port-masters must check buildable state *everyday* >> against all MI changes without review or announcement, I'll resign >> from all maintainership. Claiming "tier III since nobody seems to care" without PR is quite unfair. --- Izumi Tsutsui
Re: CVS commit: src/distrib/atari/floppies/install
On Apr 26, 3:55am, tsut...@ceres.dti.ne.jp (Izumi Tsutsui) wrote: -- Subject: Re: CVS commit: src/distrib/atari/floppies/install | christos@ wrote: | | > | It's easier to have smaller binaries (awk?) in distrib/utils? | > | > Let's see how far we are from fitting... | | All thing I know is "it works in 7.0_BETA." | | I just think "buildability fix" won't help users, | as we discussed in the past. | http://mail-index.netbsd.org/source-changes-d/2012/08/01/msg005113.html Well, it does not help being broken for months... I say tier III, since nobody seems to care. Anyway, fixed for now. christos
Re: CVS commit: src/distrib/atari/floppies/install
christos@ wrote: > | It's easier to have smaller binaries (awk?) in distrib/utils? > > Let's see how far we are from fitting... All thing I know is "it works in 7.0_BETA." I just think "buildability fix" won't help users, as we discussed in the past. http://mail-index.netbsd.org/source-changes-d/2012/08/01/msg005113.html --- Izumi Tsutsui
Re: CVS commit: src/distrib/atari/floppies/install
christos@ wrote: > Module Name: src > Committed By: christos > Date: Sat Apr 25 18:41:55 UTC 2015 > > Modified Files: > src/distrib/atari/floppies/install: Makefile list > > Log Message: > kill sed so we fit on one floppy again. What makes you think it works? http://nxr.netbsd.org/xref/src/distrib/atari/floppies/common/dot.profile?r=1.4#51 --- Izumi Tsutsui
Re: CVS commit: src/distrib/atari/floppies/install
christos@ wrote: > | > Log Message: > | > double the size of the image to fix the build > | > | AFAIK atari doesn't handle to write, load, and mount >1440KB floppy images. > | (i.e. no ustarfs support) > > How do we fix it? Can we add ustarfs support.. No idea. I don't know atari hardware/firmware specification at all except NetBSD sources. (someone who wrote usr/mdec/fdboot might know hint?) It's easier to have smaller binaries (awk?) in distrib/utils? --- Izumi Tsutsui
Re: CVS commit: src/distrib/atari/floppies/install
On Apr 26, 2:49am, tsut...@ceres.dti.ne.jp (Izumi Tsutsui) wrote: -- Subject: Re: CVS commit: src/distrib/atari/floppies/install | christos@ wrote: | | > | > Log Message: | > | > double the size of the image to fix the build | > | | > | AFAIK atari doesn't handle to write, load, and mount >1440KB floppy images. | > | (i.e. no ustarfs support) | > | > How do we fix it? Can we add ustarfs support.. | | No idea. | | I don't know atari hardware/firmware specification at all | except NetBSD sources. | (someone who wrote usr/mdec/fdboot might know hint?) | | It's easier to have smaller binaries (awk?) in distrib/utils? Let's see how far we are from fitting... christos
Re: CVS commit: src/distrib/atari/floppies/install
On Apr 26, 2:31am, tsut...@ceres.dti.ne.jp (Izumi Tsutsui) wrote: -- Subject: Re: CVS commit: src/distrib/atari/floppies/install | christos@ wrote: | | > Module Name:src | > Committed By: christos | > Date: Wed Apr 22 20:21:11 UTC 2015 | > | > Modified Files: | > src/distrib/atari/floppies/install: Makefile | > | > Log Message: | > double the size of the image to fix the build | | AFAIK atari doesn't handle to write, load, and mount >1440KB floppy images. | (i.e. no ustarfs support) How do we fix it? Can we add ustarfs support.. christos
Re: CVS commit: src/distrib/atari/floppies/install
christos@ wrote: > Module Name: src > Committed By: christos > Date: Wed Apr 22 20:21:11 UTC 2015 > > Modified Files: > src/distrib/atari/floppies/install: Makefile > > Log Message: > double the size of the image to fix the build AFAIK atari doesn't handle to write, load, and mount >1440KB floppy images. (i.e. no ustarfs support) --- Izumi Tsutsui
CVS commit: src/distrib/atari/floppies/install
Module Name:src Committed By: tsutsui Date: Sun Feb 21 20:10:52 UTC 2010 Modified Files: src/distrib/atari/floppies/install: list Log Message: Fix indent. To generate a diff of this commit: cvs rdiff -u -r1.14 -r1.15 src/distrib/atari/floppies/install/list Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
