Possible problems on deleting output files on clean

[palantir]

Possible problems on deleting output files on clean

[derekcicerone-zz]

We could probably remove the clean code but won't writing new files run into the same issue? I haven't heard of any scenarios in which it would be desirable to track the output files before, what is the use case?

[barakka]

At least in my case, writing to the file did not create any problem, while failing on the delete would stop the incremental build for any file.

With respect to why tracking generated files it is needed for two reasons:

1. our build tool-chain is maven based, and the typescript support for maven is quite reduced (there is an half decent plugin, with a some issues however). Therefore I have to ensure that my build server receives the generated output from source control, as it will not be able to regenerate it outside eclipse (in Jenkins in this case). There several ways in which I could fix, the plugin above, some ant script using tsc from the command line, etc. buy the all bring hassles.
2. I'm the only person working with typescript on my team, but the other teammates still need the generated output to the test the app in their environment. If I don't track the generated output, the only other way is to force all of my teammates to install the eclipse plugin (and node!!! By the way, it would be great if node could be in some way directly packaged with the plugin, without requiring a separate download).

[derekcicerone-zz]

Ok, I think the best thing to do is probably just remove the clean logic for now then to unblock you.

I'm not sure how we could package up node with the plug-in since it requires different binaries for each platform. I suspect it would greatly increase the size of the download. If you find any info out though, please let me know.

[barakka]

If I'm the only one having this type of problems, don't worry about changing the current behavior. I've already put deleting behind a preference in my repo, so I'm not blocked.

You might wan't to try/catch the deleting of resources in any case, just to play it safe.

[derekcicerone-zz]

I've always considered the deletion logic a bit suspect so I think it should probably go. The only cleanup we should really be doing is deleting corresponding JavaScript and source map files when a .ts file is deleted (or renamed).

[barakka]

It looks like this issue has re-emerged in last versions of the plugin, when using the "Compile all TypeScript files on build" option. This also happens with svn, and it seems limited to full builds.

[derekcicerone-zz]

What behavior are you seeing specifically?

[barakka]

We have a project with typescript activated, with only the source folder configured (all other paths are empty). We commit every file, both the .ts and the generated .js and .map.js and the workspace is clean.

We then close eclipse and restart it. As soon as eclipse completes rebuilding the project, the generated files are either marked as changed (in case of .git repo), or marked as deleted (in svn repo).

Git seems smarter, 'cos attempting a synchronize afterwards shows no changes. But the svn team provider includes the resources for deletion. Please note that the generated files are there, they are just marked as deleted (and will be deleted if one commits). Changing a source to force the update of the generated ones has the same effect, with the generated file being physically there but marked for delete.

The problem, I believe, depends on the fact that cleaning the project removes the generated files (which would be normal in the general case, but a problem in out setup), and a full build seems to issue a clean and then a compile.

Originally I had patched the plugin to delete the generated files only if the source file was deleted, but we have since then switched to the original plugin to receive the new updates.

Honestly I don't know which could be a good solution satisfying all use cases. Maybe a configuration switch that would avoid cleaning generated resources during a clean (as silly as it might be)?

[derekcicerone-zz]

For a full build, I believe that delete and rebuild is the expected behavior, right? Restarting eclipse shouldn't automatically trigger a full rebuild though - is that a workspace option perhaps?

[barakka]

Don't think so. We just have the workspace set up with "Build automatically" and I believe that triggers the build (for instance after a svn checkout).

But the problem is unrelated with re-opening eclipse. We can just commit everything, call for a full build without touching any source and we would get the generated files not only marked as modified (which we could live with) but also marked as deleted (with svn), which then generates all types of commit problems.

The problem might be with the svn team provider, that once it detects the file has been deleted it keeps it marked deleted even if regenerated. But I've not found any svn option to change this behavior.

[derekcicerone-zz]

Hmm, I'm looking at the code now and I don't see where the compiled JavaScript file would be deleted during normal compilation. They would definitely be deleted during a clean build but that should be it.