search for: checksiz

- Remove padsize argument as it is never used. - Add an usage printed when $file is not set or --help, -h is the first argument. - Add basic tests for this script. --- mbr/checksize.pl | 27 ++++++++++++++++++--------- mbr/checksize_test.sh | 22 ++++++++++++++++++++++ 2 files changed, 40 insertions(+), 9 deletions(-) create mode 100644 mbr/checksize_test.sh diff --git a/mbr/checksize.pl b/mbr/checksize.pl index 4b42327..cbca5b2 100755 --- a/mbr/checksize.pl +++ b/mbr/...

...crit : 2015-11-19 17:30 UTC+01:00, Nicolas Cornu via Syslinux <syslinux at zytor.com>: > - Remove padsize argument as it is never used. > - Add an usage printed when $file is not set or --help, -h > is the first argument. > - Add basic tests for this script. > --- > mbr/checksize.pl | 27 ++++++++++++++++++--------- > mbr/checksize_test.sh | 22 ++++++++++++++++++++++ > 2 files changed, 40 insertions(+), 9 deletions(-) > create mode 100644 mbr/checksize_test.sh > > diff --git a/mbr/checksize.pl b/mbr/checksize.pl > index 4b42327..cbca5b2 100755 >...

2015-11-19 17:30 UTC+01:00, Nicolas Cornu via Syslinux <syslinux at zytor.com>: > - Remove padsize argument as it is never used. > - Add an usage printed when $file is not set or --help, -h > is the first argument. > - Add basic tests for this script. > --- > mbr/checksize.pl | 27 ++++++++++++++++++--------- > mbr/checksize_test.sh | 22 ++++++++++++++++++++++ > 2 files changed, 40 insertions(+), 9 deletions(-) > create mode 100644 mbr/checksize_test.sh > > diff --git a/mbr/checksize.pl b/mbr/checksize.pl > index 4b42327..cbca5b2 100755 >...

...>> <syslinux at zytor.com>: >> > - Remove padsize argument as it is never used. >> > - Add an usage printed when $file is not set or --help, -h >> > is the first argument. >> > - Add basic tests for this script. >> > --- >> > mbr/checksize.pl | 27 ++++++++++++++++++--------- >> > mbr/checksize_test.sh | 22 ++++++++++++++++++++++ >> > 2 files changed, 40 insertions(+), 9 deletions(-) >> > create mode 100644 mbr/checksize_test.sh >> > > >> Just to make sure: the goal of the shell sc...

...gt;> This is not an UI. AFAIK, this script is not shipped to the user. I >> don't even see what a user could do with it. >> >> > > Well, the Perl script _is_ shipped in the official release My bad. I only checked the debian packages, none of them include a file named checksize.pl. I just checked the official release, it actually include the whole source tree. Does that mean we can't change any argument of the shell or perl scripts or any target of the Makefiles? Of course not. > >> > >> > In the particular case of "padsize" vs. "...

...r. I >> >> don't even see what a user could do with it. >> >> >> >> >> > >> > Well, the Perl script _is_ shipped in the official release >> >> My bad. I only checked the debian packages, none of them include a >> file named checksize.pl. >> I just checked the official release, it actually include the whole >> source tree. Does that mean we can't change any argument of the shell >> or perl scripts or any target of the Makefiles? >> Of course not. >> >> > >> >> > >>...

...01:00, Nicolas Cornu via Syslinux <syslinux at zytor.com>: > > - Remove padsize argument as it is never used. > > - Add an usage printed when $file is not set or --help, -h > > is the first argument. > > - Add basic tests for this script. > > --- > > mbr/checksize.pl | 27 ++++++++++++++++++--------- > > mbr/checksize_test.sh | 22 ++++++++++++++++++++++ > > 2 files changed, 40 insertions(+), 9 deletions(-) > > create mode 100644 mbr/checksize_test.sh > > > Just to make sure: the goal of the shell script is to check the per...

...h as: > > _ "ms-sys-free" (current latest code is older than its parent); > > _ a fork of "ms-sys" by Pete Batard (used in RUFUS) with additional > > recent patches. > > These projects are completely unrelated to syslinux. Should someone > use our checksize.pl, he would probably copy it to his repository. As > I said, this script isn't shipped to the user. The only purpose of > this script compared to the standard command "truncate" is that it has > several default size recorded depending on the file name pattern and > fail...

...cript is not shipped to the user. I > >> don't even see what a user could do with it. > >> > >> > > > > Well, the Perl script _is_ shipped in the official release > > My bad. I only checked the debian packages, none of them include a > file named checksize.pl. > I just checked the official release, it actually include the whole > source tree. Does that mean we can't change any argument of the shell > or perl scripts or any target of the Makefiles? > Of course not. > > > > >> > > >> > In the particula...

Hi, Martin Str|mberg wrote: > And given that space in isohdpfx is very limited (432 bytes) perhaps > the right thing might be to not have that xor in there. > But then better safe than sorry, and given that there is space _now_, > why not? I still agree. But while answering Ady's questions i get to a question of my own: Do all six fixed isohdp*.bin variations still fit into 432

...gptmbr_c.bin isohdpfx_c.bin isohdppx_c.bin \ - mbr_f.bin altmbr_f.bin gptmbr_f.bin isohdpfx_f.bin isohdppx_f.bin + mbr_f.bin altmbr_f.bin gptmbr_f.bin isohdpfx_f.bin isohdppx_f.bin \ + isohdpfv.bin isohdppv.bin %.o: %.S $(CC) $(MAKEDEPS) $(SFLAGS) -Wa,-a=$*.lst -c -o $@ $< diff --git a/mbr/checksize.pl b/mbr/checksize.pl index 4b42327..1fb2a86 100755 --- a/mbr/checksize.pl +++ b/mbr/checksize.pl @@ -27,7 +27,7 @@ if (!defined($maxsize)) { $maxsize = $padsize = 440; } elsif ($file =~ /^gptmbr[^0-9a-z]/) { $maxsize = $padsize = 440; - } elsif ($file =~ /^isohdp[fp]x[^0-9a-z]/) { +...

.../Makefile index 0bdf7e3..b9d743d 100644 --- a/mbr/Makefile +++ b/mbr/Makefile @@ -17,7 +17,7 @@ topdir = .. include $(topdir)/MCONFIG.embedded -all: mbr.bin gptmbr.bin isohdpfx.bin +all: mbr.bin gptmbr.bin isohdpfx.bin mbr-diag.bin .PRECIOUS: %.o %.o: %.S @@ -31,6 +31,10 @@ mbr.bin: mbr.elf checksize.pl $(OBJCOPY) -O binary $< $@ $(PERL) checksize.pl $@ 440 +mbr-diag.bin: mbr-diag.elf checksize.pl + $(OBJCOPY) -O binary $< $@ + $(PERL) checksize.pl mbr-diag.bin 440 + isohdpfx.bin: isohdpfx.elf checksize.pl $(OBJCOPY) -O binary $< $@ $(PERL) checksize.pl $@ 432 diff --git a/...

.../Makefile index 0bdf7e3..b9d743d 100644 --- a/mbr/Makefile +++ b/mbr/Makefile @@ -17,7 +17,7 @@ topdir = .. include $(topdir)/MCONFIG.embedded -all: mbr.bin gptmbr.bin isohdpfx.bin +all: mbr.bin gptmbr.bin isohdpfx.bin mbr-diag.bin .PRECIOUS: %.o %.o: %.S @@ -31,6 +31,10 @@ mbr.bin: mbr.elf checksize.pl $(OBJCOPY) -O binary $< $@ $(PERL) checksize.pl $@ 440 +mbr-diag.bin: mbr-diag.elf checksize.pl + $(OBJCOPY) -O binary $< $@ + $(PERL) checksize.pl mbr-diag.bin 440 + isohdpfx.bin: isohdpfx.elf checksize.pl $(OBJCOPY) -O binary $< $@ $(PERL) checksize.pl $@ 432 diff --git a/...

.../Makefile index 0bdf7e3..b9d743d 100644 --- a/mbr/Makefile +++ b/mbr/Makefile @@ -17,7 +17,7 @@ topdir = .. include $(topdir)/MCONFIG.embedded -all: mbr.bin gptmbr.bin isohdpfx.bin +all: mbr.bin gptmbr.bin isohdpfx.bin mbr-diag.bin .PRECIOUS: %.o %.o: %.S @@ -31,6 +31,10 @@ mbr.bin: mbr.elf checksize.pl $(OBJCOPY) -O binary $< $@ $(PERL) checksize.pl $@ 440 +mbr-diag.bin: mbr-diag.elf checksize.pl + $(OBJCOPY) -O binary $< $@ + $(PERL) checksize.pl mbr-diag.bin 440 + isohdpfx.bin: isohdpfx.elf checksize.pl $(OBJCOPY) -O binary $< $@ $(PERL) checksize.pl $@ 432 diff --git a/...

...> mbr/Makefile builds among other MBRs: > isohdpfx.bin isohdppx.bin isohdpfx_c.bin isohdppx_c.bin > isohdpfx_f.bin isohdppx_f.bin Yes. I make my changes in my syslinux git clone. All of the above is build from isohdpfx.S with different #defines active. Building them runs a check with checksize.pl that fails if they go beyond what's defined in checksize.pl. That does not fail for me. Thomas, if you're interested in the resulting assembly and/or encodings (as you seems to analyse and change the binary code in the resulting .bin files I assume you are) you should really clone t...

...+++ b/diag/mbr/Makefile @@ -25,7 +25,7 @@ all: handoff.bin $(CC) $(MAKEDEPS) $(SFLAGS) -Wa,-a=$*.lst -c -o $@ $< .PRECIOUS: %.elf -%.elf: %.o $(mbrdir)/mbr.ld +%.elf: %.o $(mbrdir)/$(ARCH)/mbr.ld $(LD) $(LDFLAGS) -T $(mbrdir)/$(ARCH)/mbr.ld -e _start -o $@ $< %.bin: %.elf $(mbrdir)/checksize.pl -- 2.5.3

...SSEMBLY__ -Wa,-a=isohdppx_f.lst -DFORCE_80 -c -o isohdppx_f.o /home/wferi/syslinux/syslinux/mbr/isohdppx.S ld -m elf_i386 -T /home/wferi/syslinux/syslinux/mbr/i386/mbr.ld -e _start -o isohdppx_f.elf isohdppx_f.o objcopy -O binary isohdppx_f.elf isohdppx_f.bin perl /home/wferi/syslinux/syslinux/mbr/checksize.pl isohdppx_f.bin chmod -x isohdppx_f.bin rm mbr_f.o isohdppx_c.o altmbr_c.o isohdppx_f.o gptmbr_c.o isohdppx.o altmbr_f.o altmbr.o isohdpfx_c.o mbr_c.o gptmbr_f.o gptmbr.o isohdpfx_f.o isohdpfx.o mbr.o make[3]: Leaving directory `/home/wferi/syslinux/syslinux/efi32/mbr' make -r -C sample SRC=...

I have put a compile hack in for older distros and pushed it out as syslinux-3.62-pre16; for those who have reported compilation problem on older distros I would greatly appreciate if you could confirm if it works now (it worked for me on a CentOS 3.9 VM.) -hpa

...e -D__ASSEMBLY__ -Wa,-a=isohdppx_f.lst -DFORCE_80 -c -o isohdppx_f.o /home/wferi/syslinux/genec/mbr/isohdppx.S ld -m elf_i386 -T /home/wferi/syslinux/genec/mbr/i386/mbr.ld -e _start -o isohdppx_f.elf isohdppx_f.o objcopy -O binary isohdppx_f.elf isohdppx_f.bin perl /home/wferi/syslinux/genec/mbr/checksize.pl isohdppx_f.bin chmod -x isohdppx_f.bin rm mbr_f.o isohdppx_c.o altmbr_c.o isohdppx_f.o gptmbr_c.o isohdppx.o altmbr_f.o altmbr.o isohdpfx_c.o mbr_c.o gptmbr_f.o gptmbr.o isohdpfx_f.o isohdpfx.o mbr.o make[3]: Leaving directory `/home/wferi/syslinux/genec/efi32/mbr' make -r -C sample SRC=&qu...

...erl -p -e 's/\&quot;/"/g' | perl -p -e 's/&lt;/</g' | perl -p -e 's/\&gt;/>/g' > mbr/martins_mbr.S 2017-03-21 13:18:36 dpchrist at hd-tracy ~/src/syslinux/syslinux-6.03+dfsg+dpchrist $ perl -pi -e 's/(isohdp\[fp\]x)/($1|martins_mbr)/' mbr/checksize.pl 2017-03-21 13:33:55 dpchrist at hd-tracy ~/src/syslinux/syslinux-6.03+dfsg+dpchrist $ make ... 2017-03-21 13:35:26 dpchrist at hd-tracy ~/src/syslinux/syslinux-6.03+dfsg+dpchrist $ ll bios/mbr/martins_mbr.bin -rw-r--r-- 1 dpchrist dpchrist 432 2017/03/21 13:28:30 bios/mbr/martins_mbr.bin...