adtools icon indicating copy to clipboard operation
adtools copied to clipboard

SDK 53.34 breaks native_build/makefile extraction

Open 3246251196 opened this issue 3 years ago • 1 comments

The 53.34 LHA now contains a new LHA named "execsg_sdk-54.26.lha". "base.lha" no longer contains the complete headers as it did before.

I think this section of the Makefile needs to change?

#
# Prepares the includes
#
includes-done: downloads-done
	mkdir -p $(CROSS_PREFIX)/ppc-amigaos/SDK/include
	cd downloads && lha x SDK_$(SDK_VERSION).lha
# We built clib2 inplace
#	cd downloads/SDK_Install && lha xf clib2*.lha
	cd downloads/SDK_Install && lha xf newlib*.lha
	cd downloads/SDK_Install && lha xf base.lha
	cd downloads/SDK_Install && rm -Rf *.lha
	cd downloads/SDK_Install && mv newlib* $(CROSS_PREFIX)/ppc-amigaos/SDK
#	cd downloads/SDK_Install && mv clib2* $(CROSS_PREFIX)/ppc-amigaos/SDK
	cd downloads/SDK_Install && mv Include/* $(CROSS_PREFIX)/ppc-amigaos/SDK/include
	rm -Rf downloads/SDK_Install downloads/SDK_Install.info
	touch $@

I will get around to making a pull request at some point. Just not now. And, all that is on the assumption that my report is correct.

To reproduce: CLONE this REPO Hack in the new link to the 53.34 SDK download link instead of getting 53.30 and "make native-build"

Notice that there is a compilation error due to no "exec/types.h".

Of course, I guess using 53.34 is not supported anyway at the moment in this repo. So, probably updating to support 53.34 would include this fix.

3246251196 avatar Jan 08 '22 21:01 3246251196

Another really important thing to remember is the version of LHA you are running on your non-Amiga machine; in my case some ancient x86_64 running Ubuntu 20. The version of LHA from the "lhasa" package will just stop extracting once it sees an empty directory entry (which happens in base.lha of the 53.34 SDK). I decided to compile: https://github.com/jca02266/lha and that one works.

3246251196 avatar Jan 09 '22 10:01 3246251196

Suggested status for this ticket: wait for latest SDK which will make public Reschedule() (Exec) again, and we will have to see if we still need to extract execsg separately in the makefile.

3246251196 avatar Oct 05 '22 11:10 3246251196

@sba1 This ticket can be closed.

3246251196 avatar Oct 09 '22 16:10 3246251196

"

3246251196 avatar Oct 09 '22 16:10 3246251196