OrangeC icon indicating copy to clipboard operation
OrangeC copied to clipboard

removed OCIDE from installer

Open GitMensch opened this issue 1 year ago • 1 comments

fixes #1020

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

warning: untested...

GitMensch avatar Feb 06 '24 07:02 GitMensch

@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 avatar Feb 09 '24 23:02 LADSoft

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

LADSoft avatar May 13 '24 19:05 LADSoft

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

GitMensch avatar May 14 '24 08:05 GitMensch

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 avatar May 14 '24 17:05 LADSoft

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

LADSoft avatar May 14 '24 22:05 LADSoft