Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] Language Settings and the new build system

That's what I'm trying. I also have adapters back and forth between them. Seems to be working. I need to find the right place to create the IBuildConfiguration for new configuration descriptors. Doing it lazily caused workspace locked exceptions.

This strategy might also allow me to leverage the environment variable manager.

Love to hear any feedback before I go to far.

Doug.

On Tue, Mar 15, 2016 at 1:03 PM, Sergey Prigogin <eclipse.sprigogin@xxxxxxxxx> wrote:
Does this mean that the type of the first parameter in ILanguageSettingsProvider.getSettingEntries will be changed from ICConfigurationDescription to IBuildConfiguration?

On Tue, Mar 15, 2016 at 7:02 AM, Doug Schaefer <cdtdoug@xxxxxxxxx> wrote:
OK, that seemed to be a good direction. Seems like the config description is just passed around for the most part.

Now, I'm going to get brave and use IBuildConfiguration instead. We'll create these things on the fly as needed and provide an adapter back to ICConfigurationDescription. Wish me luck :).

Doug.

On Mon, Mar 14, 2016 at 3:12 PM, Doug Schaefer <cdtdoug@xxxxxxxxx> wrote:
The other option would be to pass configuration id's around as Strings. They easily map to IBuildConfiguration, project.getBuildConfig(String). They providers could try and map them to a ICConfigurationDescription to get info out of there, or adapt them to the new CBuildConfiguration to get stuff out of there.

On Mon, Mar 14, 2016 at 2:58 PM, Sergey Prigogin <eclipse.sprigogin@xxxxxxxxx> wrote:
It's worth noting that LanguageSettingsSerializableProvider.getSettingEntries(ICConfigurationDescription, IResource, String) doesn't use its first parameter. One possible approach is to replace ICConfigurationDescription in ILanguageSettingsProvider.getSettingEntries with ICSettingContainer and to add instanceof checks in classes like ScannerInfoExtensionLanguageSettingsProvider and PathEntryScannerInfoLanguageSettingsProvider that depend on ICConfigurationDescription.

-sergey

On Mon, Mar 14, 2016 at 11:20 AM, Doug Schaefer <cdtdoug@xxxxxxxxx> wrote:
One of the actions I took back from the CDT summit was to explore how to use the Language Settings Provider with it. I think we agree that we do need the same functionality that it provides for all types of projects.

My initial investigation, though, shows that the provider system is intrinsically tied to the project descriptor system, and ICConfigurationDescription in particular. Almost all APIs reference it.

The new system is based on the platform resource's IBuildConfiguration. It appears to me that I will probably have to duplicate the entire language settings provider system to work in that new architecture. I'm OK with that. I just wanted to give people a heads up and see if anyone can think of a better approach.

The only one I can think of is to bridge ICConfigurationDescriptor to IBuildConfigurations, but that wouldn't work for the new style projects as I want to avoid the old project descriptor system.

Doug.

_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cdt-dev


_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cdt-dev



_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cdt-dev


_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cdt-dev


Back to the top