Skip to main content


Eclipse Community Forums
Forum Search:

Search      Help    Register    Login    Home
Home » Eclipse Projects » 4DIAC - Framework for Distributed Industrial Automation and Control » Changes in forte_sync.cpp
Changes in forte_sync.cpp [message #1798440] Mon, 19 November 2018 09:01 Go to next message
Wolfgang Trischnigg is currently offline Wolfgang TrischniggFriend
Messages: 34
Registered: April 2016
Member
As I was getting errors when I build the forte, I have to change this line in the forte_sync.cpp

CPCSyncObject::CPCSyncObject() {
InitializeSRWLock(&mLock);
}

Can you please add this to Release 1.10.?

Best,
Wolfgang
Re: Changes in forte_sync.cpp [message #1798441 is a reply to message #1798440] Mon, 19 November 2018 09:09 Go to previous messageGo to next message
Martin Melik-Merkumians is currently offline Martin Melik-MerkumiansFriend
Messages: 74
Registered: March 2016
Member
In order to be safe from the legal perspective, please create an Eclipse Account and sign an Eclipse Contributors Agreement.
After you have done this, you can push your change to the Gerrit code review system. For this you can either use the git review command, if you have installed it, or stick to the documentation found at https://www.eclipse.org/4diac/en_help.php "Developmen Documentation"->"FORTE"->"Contributing to FORTE"

If you want to use git review, install it, then in the FORTE repository type the command "git review -s" to load the setup from the repository. After that do your changes, add and commit them, and at last, if you want to push them to the 1.10.x code review, type "git review 1.10.x".

Please also create a bug and explain your issue, as each platform has its own forte_sync.cpp

Best regards,
Martin
Re: Changes in forte_sync.cpp [message #1798442 is a reply to message #1798440] Mon, 19 November 2018 09:13 Go to previous messageGo to next message
Jose Maria Jesus Cabral Lassalle is currently offline Jose Maria Jesus Cabral LassalleFriend
Messages: 160
Registered: February 2016
Senior Member
Can you tell us which platform and compiler version are you using? If it's windows and mingw, check this https://www.eclipse.org/forums/index.php/t/1096099/
Re: Changes in forte_sync.cpp [message #1798550 is a reply to message #1798442] Tue, 20 November 2018 10:47 Go to previous messageGo to next message
Wolfgang Trischnigg is currently offline Wolfgang TrischniggFriend
Messages: 34
Registered: April 2016
Member
Hello I'm using:
- WIN32
- VisualStudio 2010 Express
- CMake 3.1.1
Re: Changes in forte_sync.cpp [message #1798578 is a reply to message #1798550] Tue, 20 November 2018 15:38 Go to previous messageGo to next message
Martin Melik-Merkumians is currently offline Martin Melik-MerkumiansFriend
Messages: 74
Registered: March 2016
Member
Which exact Windows version do you use? Your error is most likely related to the OS API for thread synchronization, which is dependent on the exact version of your OS.
Re: Changes in forte_sync.cpp [message #1798580 is a reply to message #1798578] Tue, 20 November 2018 15:58 Go to previous messageGo to next message
Wolfgang Trischnigg is currently offline Wolfgang TrischniggFriend
Messages: 34
Registered: April 2016
Member
Windows 10 Pro,
Version 1803,
Systembuild 17134.407
Re: Changes in forte_sync.cpp [message #1798622 is a reply to message #1798580] Wed, 21 November 2018 09:38 Go to previous message
Martin Melik-Merkumians is currently offline Martin Melik-MerkumiansFriend
Messages: 74
Registered: March 2016
Member
Ok, as I also run even the same build of Win 10 Pro, I would guess that the API delivered with Visual Studio changed some when between 2010 and 2017.
Previous Topic:ID data format and SD/RDs config for MQTT communication
Next Topic:Control System app for Beckhoff Using OPC-UA
Goto Forum:
  


Current Time: Fri Nov 15 15:33:22 GMT 2019

Powered by FUDForum. Page generated in 0.01729 seconds
.:: Contact :: Home ::.

Powered by: FUDforum 3.0.2.
Copyright ©2001-2010 FUDforum Bulletin Board Software

Back to the top