search

LADSoft / OrangeC

OrangeC Compiler And Tool Chain
http://ladsoft.tripod.com/orange_c_compiler.html
Other
284 stars 39 forks source link

removed OCIDE from installer #1021

Closed GitMensch closed 1 month ago

GitMensch commented 5 months ago

fixes #1020

additionally moved the examples to its own installer option, installing it directly below OrangeC

warning: untested...

LADSoft commented 5 months ago

@GitMensch

there is a problem with these commits: you've changed various CPJ files to refer to where the files exist in the source tree, however, the files don't get installed there by occ.iss. But that speaks to a broader issue because the files are installed in one place for desktop apps and in another place for thumbdrive apps, but, the CPJ files have fixed paths in them. Possibly, the CPJ files need relative paths (from the location of the CPJ file) to take care of all three use cases, which I think is something we added to later versions of OCIDE but don't remember for sure...

LADSoft commented 1 month ago

im doing the gitignore and dosxclude.lst separately in another commit, I'm going to close this one

GitMensch commented 1 month ago

e340548d8247a24b2b0b615045c4b90bc06c3593 handled the two files - but the removal of the OCIDE parts and the adjustment for the installer concerning the examples are still missing.

LADSoft commented 1 month ago

i guess i missed that this also changed the iss file. But the reason i didn't accept this pull request is that as i commented earlier, it broke the builds. I think the problem was related to moving the examples.... so I didn't even try to do it.... I'll change the ISS file later on today.

LADSoft commented 1 month ago

so will fix this and the examples thing in conjunction with closing #1020...