[le717]

Towards #8751.

Removes APIs listed in #6127. As you can see in that issue, there are a number of extensions that access it, but it's nearly a year since they were deprecated. It's time to remove them.

[le717]

This was a part of the auto-install extensions PR, but was reverted during review. Since I was editing this file, I thought I would go ahead and restore it.

[peterflynn]

Fwiw, there's basically zero chance that this will make any performance difference. But it's not a big deal to change either way.

[le717]

I should note I have already contacted all affected extension authors about the deprecation and eventual removal.

[peterflynn]

@le717 Here's my list of all the extensions that will break with this PR:

• (filed) brackets-docco - FileIndexManager, NativeFileSystem
• brackets-typescript-code-intel - FileIndexManager, NativeFileSystem
• jhatwich.brackets-related-files - FileIndexManager
• (filed) aem-sightly-brackets-extension - NativeFileSystem
• (filed) alexsalas.brackets-jekyll - NativeFileSystem
• (filed) brackets-bower - NativeFileSystem
• brackets-minify-more - NativeFileSystem
• cfbrackets.org - NativeFileSystem
• dehats.appcachebuddy - NativeFileSystem
• (filed) dehats.ccextbuilder - NativeFileSystem

Can you verify that you've filed bugs with all of them already?

[edit: I added "(filed)" next to the ones where I see bugs linked back to #6127 -- for the others I'm not sure whether any bug has been filed yet]

[le717]

@peterflynn It looks like I've issues for most of these, unless GitHub is not showing some of the link backs. Will file the remaining.

brackets-docco - FileIndexManager, NativeFileSystem
brackets-typescript-code-intel - FileIndexManager, NativeFileSystem
jhatwich.brackets-related-files - FileIndexManager (jhatwich/brackets-related-files#3)
alexsalas.brackets-jekyll - NativeFileSystem
brackets-bower - NativeFileSystem
brackets-minify-more - NativeFileSystem
cfbrackets.org - NativeFileSystem
dehats.appcachebuddy - NativeFileSystem (davidderaedt/appcache-gen#1)
dehats.ccextbuilder - NativeFileSystem

[le717]

@peterflynn I've contacted the rest of the extension authors. A few of them already had been filed (as denoted by the link to it), and I followed up on the report.

• brackets-typescript-code-intel I personally submitted PRs to fix any deprecated code, but they have not been pushed to the registry because of another broken part.
• brackets-minify-more is a fork of https://github.com/wylst/brackets-minifier. The upstream version has long been updated, but as the fork has issues I cannot directly file a report. @Fornace Would you mind looking into this pending removal and updating your extension appropriately?
  * cfbrackets.org Appears to have already been fixed from Deprecated filesystem APIs raeberli/cfbrackets#1. @raeberli Can you confirm this deprecation will not effect your extensions?

[le717]

@peterflynn Do you still want to get this into 1.1? I want to avoid a repeat of the 1.0 issues as much as you do, and if 1.1 is supposed to be released near December 9, the sooner this lands the better chance we have of less extension breakage.

[raeberli]

cfbrackets has no reference to NativeFileSystem anymore since 7th jun

From: Triangle717 [mailto:notifications@github.com]
Sent: Samstag, 15. November 2014 18:03
To: adobe/brackets
Cc: raeberli
Subject: Re: [brackets] Remove deprecated file system APIs (#9622)

@peterflynn https://github.com/peterflynn I've contacted the rest of the extension authors. A few of them already had been filed (as denoted by the link to it), and I followed up on the report.

• brackets-typescript-code-intel I personally submitted PRs to fix any deprecated code, but they have not been pushed to the registry because of another broken part.
• brackets-minify-more is a fork of https://github.com/wylst/brackets-minifier. The upstream version has long been updated, but as the fork has issues I cannot directly file a report. @Fornace https://github.com/Fornace Would you mind looking into this pending removal and updating your extension appropriately?
• cfbrackets.org Appears to have already been fixed from Deprecated filesystem APIs raeberli/cfbrackets#1 Deprecated filesystem APIs raeberli/cfbrackets#1 . @raeberli https://github.com/raeberli Can you confirm this deprecation will not effect your extensions?

—
Reply to this email directly or view it on GitHub #9622 (comment) . https://github.com/notifications/beacon/ACEfRgqF6tHrMgsYqb6EwlXpsbUTKvm4ks5nN360gaJpZM4CxWkQ.gif

[le717]

@raeberli Thanks for the update. :)

[le717]

This PR has been coming up on a year open and two years since the old file system was deprecated. Any extensions that are not updated by now will probably never be updated. Could someone perhaps look at this and see about getting it merged?

[zaggino]

Agreed @le717 , lets clear this. Merging into master, will take effect from 1.6 (1.5 is now on release branch)