Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MSFS native support #266

Open
wants to merge 36 commits into
base: main
Choose a base branch
from
Open

MSFS native support #266

wants to merge 36 commits into from

Conversation

tzobler
Copy link
Contributor

@tzobler tzobler commented Feb 28, 2024

All necessary fields in the GUI and mapping tool have been created (based on Roland's changes). Tested communication with datastore and no errors so far. "Saved settings" message added when saving distributor selection. Search for aircaft.cfg optimized for MSFS

@tzobler tzobler added the MSFS label Feb 28, 2024
@tzobler tzobler self-assigned this Feb 28, 2024
@CLAassistant
Copy link

CLAassistant commented Feb 28, 2024

CLA assistant check
All committers have signed the CLA.

@tzobler tzobler linked an issue Feb 29, 2024 that may be closed by this pull request
4 tasks
src/blackcore/fsd/fsdclient.cpp Outdated Show resolved Hide resolved
@@ -38,6 +38,9 @@ namespace BlackConfig
//! with FG support?
static constexpr bool isCompiledWithFGSupport(); // defined in buildconfig_gen.inc.in

//! with MSFS support?
static constexpr bool isCompiledWithMSFSSupport(); // defined in buildconfig_gen.inc.in
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

More a general thing: should we rename MSFS internally to MSFS2020 or something similar? Otherwise it gets confusing because we already use "MsFlightSimulator" (which could also be abbreviated as MSFS) at other place. And how would we call it when MSFS 2024 will be released 😉

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest MSFS20 and later MSFS24. Agreed?

resources/share/shared/bootstrap/bootstrap.json Outdated Show resolved Hide resolved
@@ -38,6 +38,9 @@ namespace BlackConfig
//! with FG support?
static constexpr bool isCompiledWithFGSupport(); // defined in buildconfig_gen.inc.in

//! with MSFS support?
static constexpr bool isCompiledWithMSFSSupport(); // defined in buildconfig_gen.inc.in

//! with FSUIPC support?
static constexpr bool isCompiledWithFsuipcSupport(); // defined in buildconfig_gen.inc.in

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should MSFS 2020 also be added to the method on line 48 (isCompiledWithMsFlightSimulatorSupport()?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, I'll upload it with the next PUSH

src/blackconfig/buildconfig_gen.inc.in Outdated Show resolved Hide resolved
//! Find the config files (fsx.cfg)
// C:/Users/Joe Doe/AppData/Roaming/Microsoft/FSX/fsx.cfg
static QStringList findFsxConfigFiles();

// TODO TZ
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this TODO still outstanding?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this function doesn't do what it's supposed to do. That's why I want to watch it

//! Get all the SimObjects paths from fsx.cfg
// SimObjectPaths.0=SimObjects\Airplanes
static QSet<QString> fsxSimObjectsPaths(const QStringList &fsxFiles, bool checked);

// TODO TZ
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this TODO still outstanding?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this function doesn't do what it's supposed to do. That's why I want to watch it

//! Get all the SimObjects files from fsx.cfg
// SimObjectPaths.0=SimObjects\Airplanes
static QSet<QString> fsxSimObjectsPaths(const QString &fsxFile, bool checked);

// TODO TZ
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this TODO still outstanding?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this function doesn't do what it's supposed to do. That's why I want to watch it

Comment on lines +184 to +186
// todo msfs: MSFS specific model validation
// const CStatusMessageList specificTests2 = FsCommon::CFsCommonUtil::validateMSFSSimObjectsPath(models, validModels, invalidModels, ignoreEmpty, stopAtFailedFiles, wasStopped, simulatorDir);
// specificTests.push_back(specificTests2);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Open a GitHub issue for this, so it does not get lost.

src/blackgui/components/dblogincomponent.ui Outdated Show resolved Hide resolved
@tzobler tzobler removed the request for review from oktal3700 March 12, 2024 20:42
@tzobler tzobler requested a review from ltoenning March 28, 2024 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Natively support Microsoft Flight Simulator 2020 (MSFS)
4 participants