remy / vscode-nextbasic

VS Code tools for NextBASIC
https://marketplace.visualstudio.com/items?itemName=remysharp.nextbasic
9 stars 2 forks source link

VSCode Extension still doesn't work... #9

Closed Andy1966uk closed 4 years ago

Andy1966uk commented 4 years ago

Looks like two issues, 1. Programs don;t automatically run any more. CSPECT used to launch and run program, now you have to manually go into NEXTBasic (where you see your program listing#) and RUN it.

  1. Assets no longer get ported over to the CSPECT next image... only index.bas gets copied over into the devel folder
remy commented 4 years ago

Can you confirm the extension version you're on?

remy commented 4 years ago

I've just tested with windows and the latest 1.3.8 extension and the program is autorunning (I've not been able to check assets yet).

However I am seeing this bug: #6, so I tested with

10 PRINT "Hello"
20 PRINT "World"
30 GO TO 10
jsanjose commented 4 years ago

No problem here. The extension copies all files and folders present inside the folder you are running the Basic program. And also "autolaunch" the tokenized file (index.bas)

image

Perhaps, are you using Windows and path with spaces?

Andy1966uk commented 4 years ago

Not working for me on 1.3.8 or 1.3.9 program don;t autorun and assets aren't copied over, only the program which gets renamed " index.bas"

remy commented 4 years ago

Sod. I'm pretty sure the code by itself is sound (as I did a fresh install), but there's something else at play.

Can I ask:

I'm not sure that will bring anything up, but it's a start.

Andy1966uk commented 4 years ago

Working with previous codebase yes New Directory work with autorun and porting assets to image ok have tried uninstalling and reinstalling extension Continuing to test on current folders and files now..... will let you know

Andy1966uk commented 4 years ago

Old Folder and Files don't work.... will try backing up, deleting and creating new folder and workspace structure, maybe that will work...

Andy1966uk commented 4 years ago

Tried renaming folder, resetting and reinstalling Extension, brand new folder and contents worked though, but on all older programs assets don;t get copied over. Also removed the devel folder and all contents....

Andy1966uk commented 4 years ago

I have noticed a .vscode folder appears in the new folder though, but is not present in the other folders... could that be it ?

remy commented 4 years ago

that's your vscode editor putting that there, but the push process skips over files prefixed with a ..

Okay, so we know it's something to do with the setup of your existing codebase and how the extension is using it.

On an older project (that doesn't work), can you open vscode and then from Help->Toggle developer tools, then do the "run with cspect" - see if anything mentions either cspect, hdfmonkey or the nextbasic extension in there?

Andy1966uk commented 4 years ago

Good, news, I enabled the development tool and it showed it was trying to copy 96 files over, and hence failed... not sure why it worked last night, but I have been backing up files in same structure, i.e variants of .spr and .map files etc. Anyway I have reduced the file number down and it works.... sorry to waste your time... thx

remy commented 4 years ago

Ah, I should really alert when that blocks out - I had noticed my own dev, I'd drop files in random places and accidentally move several gig of files into the cspect image! So I added a check that topped out 50 files. I'll change it so it's not just a silent error but a popup warning (like when export works).

Glad it's sorted :phew:!

remy commented 4 years ago

Just to let you know I've added a couple of additional warning messages when things go wrong. The upload (to cspect) limit is set to 50 and if there's more files attempting to copy, it says that it aborted in vscode (the 50 is also user configurable, so you can up this to your liking).

There's also now a warning if export fails due to validation errors and a warning (but still runs) if there's validation errors before a cspect launch.