DouglasHeriot / ember-plus-googlecode

Automatically exported from code.google.com/p/ember-plus
0 stars 0 forks source link

Compilation troubles of TinyEmberPlus on Linux #29

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
- There are sourcefiles containing BACKSLASHES in include statements (not linux 
conform)

find . -iname "*.c*" -or -iname "*.h*" -exec grep -l -e '\#[ \t]*include .*\\' 
{} \;
shows following files

- ./tinyember/TinyEmberPlusRouter/TinyEmberPlusRouter/net/TcpClient.h
- ./tinyember/TinyEmberPlus/EditEnumerationDialog.h
- ./tinyember/TinyEmberPlus/TinyEmberPlus.h
- ./tinyember/TinyEmberPlus/net/TcpClient.h
- ./tinyember/TinyEmberPlus/gadget/Subscriber.h
- ./tinyember/TinyEmberPlus/GadgetViewContextMenu.h
- ./tinyember/TinyEmberPlus/glow/Encoder.h
- ./tinyember/TinyEmberPlus/CreateParameterDialog.h
- ./tinyember/TinyEmberPlus/serialization/detail/GadgetTreeReader.h
- ./libformula/Headers/formula/Term.hpp
- ./libformula/Headers/formula/Error.hpp
that can be fixed with:
find . -iname "*.c*" -or -iname "*.h*" -exec sed -e '/\#[ \t]*include/ 
s/\\/\//g' {} \;

- In the provided .PRI file, files are missing:
Please add BooleanView.ui to the FORMS

- All #pragma warnings related to MSVC need to be embraced in #ifdef _MSC_VER 
... #endif statements
- a statement like (TinyEmberPlus/gadget/Node.h):
typedef DirtyStateListener<NodeFieldState::flag_type, Node const*> 
DirtyStateListener;
will lead to heaps of warnings and errors using GCC 4.8.2 (Yes , c++11 is 
enabled)
I had to change all of them to something like
typedef DirtyStateListener<NodeFieldState::flag_type, Node const*> 
DIRTYSTATELISTENER;

- I had to build a 'real' .PRO file from scratch to match Qt 5.3.x (attached 
for info)

Original issue reported on code.google.com by nullable...@gmail.com on 2 Dec 2014 at 8:35

GoogleCodeExporter commented 9 years ago

Original comment by nullable...@gmail.com on 2 Dec 2014 at 8:36

Attachments:

GoogleCodeExporter commented 9 years ago
Having this tool integrated into the CI, seems to be a must for me.

Original comment by nullable...@gmail.com on 3 Dec 2014 at 8:07