summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Exclude from Mac build.android-cts-6.0_r2android-cts-6.0_r1android-6.0.1_r9android-6.0.1_r8android-6.0.1_r7android-6.0.1_r5android-6.0.1_r4android-6.0.1_r3android-6.0.1_r10android-6.0.1_r1android-6.0.0_r7android-6.0.0_r6android-6.0.0_r5android-6.0.0_r41android-6.0.0_r4android-6.0.0_r3android-6.0.0_r26android-6.0.0_r25android-6.0.0_r24android-6.0.0_r23android-6.0.0_r2android-6.0.0_r13android-6.0.0_r12android-6.0.0_r11android-6.0.0_r1Jean-Michel Trivi2015-07-161-2/+2
| | | | | | | The code doesn't build on Mac. Temporarily exclude this project from Mac builld. Change-Id: Ibfd8a3ca804f463425247f27c709cf46be6ffdd8
* Drop release v2.6.0+no-stlportJean-Michel Trivi2015-07-1580-1229/+3588
| | | | | | | Bug 246391 Change-Id: I662b7b0f90c97cb169978e1b64ad1fe32c440cf5 Signed-off-by: Jean-Michel Trivi <jmtrivi@google.com>
* Merge "Exclude from Mac build." into mnc-devYing Wang2015-06-091-2/+2
|\
| * Exclude from Mac build.Ying Wang2015-06-091-2/+2
| | | | | | | | | | | | | | | | | | The code doesn't build on Mac. Temporarily exclude this project from Mac builld. Bug: 21669400 Bug: 21734546 Change-Id: I68278d69803219dadd27487ed06c52d77a5a9833
* | Fix Mac build.Ying Wang2015-06-082-50/+0
|/ | | | | | | | | It's an error to build .a without source file on Mac. These two empty static libraries aren't used anywhere. Bug: 21669400 Change-Id: I73e0899378256ddf802eb7a3aafdbe43559baefe (cherry picked from commit f1ef7259741bdf99e2a5991bc12aa6cf4f551785)
* Fix build from stlport.Dan Albert2015-05-076-7/+2
| | | | | | | | | | | | This will need to be submitted upstream, but for now it's blocking the build. Can't test yet as the build is still red, but there's no risk doing this for now because none of these modules are used yet. Bug: 20915699 Change-Id: I9e7c8e237bec134985735dee4b4225b29793a792 (cherry picked from commit 6368380f06a48b0049118140f6ac5059f4e7f0f0)
* Fix mac build: do not hardcode linux pathsJean-Michel Trivi2015-04-281-2/+2
| | | | Change-Id: I70996fcbc0c0d60c51fde207dfd8cae25265c79b
* python bindings: activate thread-safety code generationDavid Wagner2015-04-241-1/+1
| | | | | | | | | | | | | | | | | The Parameter Framework may spawn a thread listening to request for command execution. In this case, whenever a command produces a log, a callback function provided by the client is called. When the client is a python script using the python bindings, this causes a python function to be called in a different context than the main thread. This can only work safely if a specific lock is held before hand; see https://docs.python.org/2/c-api/init.html#non-python-created-threads Fortunately, SWIG can generate the corresponding code automatically using %module(threads=1, ...) mymodule Signed-off-by: David Wagner <david.wagner@intel.com>
* xml coverage: fix a copy-paste mistakeDavid Wagner2015-04-241-2/+2
| | | | | | | | | | | | Commit ccb164c15d0f46fb233b67075f37bede8f8ad855 introduced the possibility to ignore changes to unknown criterion through an option passed to coverage.py but a copy-paste mistake swapped the action of this option with the option ignoring incoherent criterion state changes. Both flags are set by 'aplog2coverage.sh -f' so the mistake went unnoticed but it must be fixed nevertheless. Signed-off-by: David Wagner <david.wagner@intel.com>
* travis: don't send email notificationsDavid Wagner2015-04-241-2/+0
| | | | | | | I never received any email from travis and actually don't want to. Thus, I'm removing my email address from the notification list. Signed-off-by: David Wagner <david.wagner@intel.com>
* travis: Use the docker infrastructureDavid Wagner2015-04-241-36/+32
| | | | | | | | | | | | | | | | | As advertise by travis news: http://blog.travis-ci.com/2014-12-17-faster-builds-with-container-based-infrastructure/ using the new docker infra is faster and more reliable. This comes with the cost of not being able to use "sudo", though. Installing packages is still possible by specifying them in an "addons.apt" section in .travis.yml. We have to remove "sudo make install" but we aren't using the installed libs and binaries anyway. I had some syntax issues with the "addons" and "env" sections and had to remove the coverity subsection which wasn't configure for the 01org repository anyway. Signed-off-by: David Wagner <david.wagner@intel.com>
* xmlcoverage: be able to ignore changes to unknown criteriaDavid Wagner2015-04-242-1/+23
| | | | | | | | | | | | | | | | Since a parameter-framework client may set a criterion before calling start(), the logs may show changes to criteria before the coverage script knows the list of criteria. It used to cause an unrecoverable error even with the "--force" option. This option now makes the coverage script ignore such errors. The script already takes the initial values of criteria into account when the parameter-framework starts, so there isn't any information loss. However, if this error happens for any other error than described in the first paragraph, you probably don't want to silence it. Signed-off-by: David Wagner <david.wagner@intel.com>
* python: prefer 2.7.x and make sure interpreter and libs are alignedDavid Wagner2015-04-241-2/+5
| | | | | | | | | | | CMake's find_package command makes it possible to specify a version. Take advantage of that to favor the interpreter's version to 2.7 and then pin the library's version to the very same version as the interpreter. This does not ensure that 2.7.x is used (e.g. if 2.7.x is not present, another version will be used) but it helps a lot. Signed-off-by: David Wagner <david.wagner@intel.com>
* Version 2.4.3David Wagner2015-04-241-1/+1
| | | | Signed-off-by: David Wagner <david.wagner@intel.com>
* Revert "Revert "Merge pull request #88 from krocard/improve_travis_debug_build""Kevin Rocard2015-04-241-8/+7
| | | | | | | | | The pull request #88 was reverted as it broke travis build. In fact it only reveled a bug that was introduced in #80 (1b071fadd). The previous commit 0c0cc95 fixes the problem (cmake variable shadow). Revert the revert (d17b931a3dcc70cf9e) to restore the pull request #88.
* Do not use MEMORYCHECK_SUPPRESSIONS_FILEKevin Rocard2015-04-241-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The pull request #88 was reverted as it broke travis build. ctest uses a cache variable MEMCHECK_SUPPRESSION_FILE. It sets it during the `include(ctest)` evaluation. Cache variable are set on creation, they can not be modify after by a CMakeList.mk (except with set's FORCE option). ef5736eb5 sets MEMCHECK_SUPPRESSION_FILE as a normal variable. This does not set the cache variable as expected but rather create a second local variable (not part of the cache) with the same name. If cmake was run once (as during travis build since patch 4e41ff7, until its revert d17b931), the local variable has no effect thus the suppression file was never saved in the cache. If cmake was run twice, during the second time, as the cache version is not recreated (cache variables are persistent), the call to set MEMCHECK_SUPPRESSION_FILE changes the cache version as it was first expected. As a result the build worked if cmake was invoked twice but failed if only called once. For more information, see the *** Variable types in CMake *** section at: http://www.cmake.org/cmake/help/v2.8.11/cmake.html#command%3aset This patch fixes all that by not using MEMCHECK_SUPPRESSION_FILE. but rather setting directly the option in valgrind. It is a clean fix as the MEMCHECK_SUPPRESSION_FILE is an option for the user to add a valgrind suppression file, it is not intended to be used by the project direcly. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
* Revert "Merge pull request #88 from krocard/improve_travis_debug_build"David Wagner2015-04-241-7/+8
| | | | | This reverts commit 339779ac82acb755f6e992bf7ee54e3b791a9ca6, reversing changes made to 1b071faddaf8c5e835ea38d6c8d2d5db88011251.
* [Travis] No longer install debug buildKevin Rocard2015-04-241-8/+7
| | | | | | | | | | | | | | | | | | | Debug build used to be installed as it was required for make test. As make test is now stand alone, do not install debug builds. This has several advantages: - slightly faster builds. - make test without install is actually tested by travis. Debug binaries used to overwrite release ones that were installed presciently. As it is no longer the case, and to avoid tests to use installed binaries instead of the local ones, test the debug build *before* the release one. As the debug build is now build (first) in it's own folder, debug and release build can not interfere. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
* [Travis] Use gcov long optionsKevin Rocard2015-04-241-1/+1
| | | | Long option are more readable.
* Install bash completions in /usr/shareKevin Rocard2015-04-241-1/+1
| | | | | | | | | | | | | | With a classic cmake install prefix '/usr', bash completions were installed in /usr/etc/bash_completion.d/ which even before bash-completion 1.9 was incorrect. Since v1.9 all completions are to be installed in /usr/share/bash-completion/completions See: http://anonscm.debian.org/cgit/bash-completion/bash-completion.git/tree/CHANGES#n324 Change the completions install path accordingly
* [Travis] Only run gcov through coverallsKevin Rocard2015-04-241-1/+1
| | | | | | | | | | Gcov was run twice, first by the target ExperimentalCoverage and then by coveralls. As coveralls give a better coverage report and cdash is not used, remove ExperimentalCoverage. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
* Fail memcheck target on leak or memory errorsKevin Rocard2015-04-242-0/+25
| | | | | | | | | | | Some unit test were checked using valgrind, but if an error was detected it was only logged: the build was not failed. Fail to build all *MemCheck targets if a leak or an error is reported by valgrind. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
* Add comments in ctest list fileKevin Rocard2015-04-241-0/+4
| | | | | Mainly add a some documentation about the custom cmake set_test_env function.
* Move ctest definitions in it's own CMakeListKevin Rocard2015-04-243-11/+42
| | | | | | | | | | Ctest related files and definitions were at the source root although we try to keep it simple. Move all ctest related files and definition in a ctest directory. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
* Move tests inclusion in a test CMakeListKevin Rocard2015-04-242-5/+34
| | | | | | | | | The root CMakeList was including every test directories directly, although it should be kept simple. Move the inclusions to a dedicated CMakeList in test. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
* Remove unused CXmlElement::setComment methodDavid Wagner2015-04-242-6/+0
| | | | Signed-off-by: David Wagner <david.wagner@intel.com>
* Remove unsued CElement::findAscendantOfKind methodDavid Wagner2015-04-242-15/+0
| | | | Signed-off-by: David Wagner <david.wagner@intel.com>
* Remove unused CElement::getLastChild methodDavid Wagner2015-04-242-10/+0
| | | | Signed-off-by: David Wagner <david.wagner@intel.com>
* coveralls: send coverage report to coveralls.ioDavid Wagner2015-04-242-0/+14
| | | | | | | | | | | | | | | coveralls.io is a service that uses information from gcov (generated during "make test") to provide a coverage report online: https://coveralls.io/r/01org/parameter-framework Some files must be ignored, specifically: generated files (python bindings and files generated by CMake), tests (because they would inflate the coverage rate) and samples (for now, only the skeleton subsystem). A "badge" is also added to the readme showing the coverage rate. Signed-off-by: David Wagner <david.wagner@intel.com>
* stop messing around with useless pointerSébastien Gonzalve2015-04-242-24/+11
| | | | | | | | | | Backsynchronizer is dynamically allocated for simulation purpose. Nevertheless, this is useless as this can be easily achieved thanks to a typedef. This patch simplify the backsynchronization use. Signed-off-by: Sébastien Gonzalve <oznog@users.noreply.github.com>
* Fix for BackSynchronizer related crash on windowsPatrick Benavoli2015-04-241-1/+1
| | | | | | | | | On windows platform, the definition of a function pointer to a member of a given class requires visibility on the header of that class, otherwize memory corruption may occur at deletion time. Change-Id: Ifa4943ef0f3e02542c6641959cf67384103bfc69 Signed-off-by: Patrick Benavoli <patrick.benavoli@intel.com>
* Inconsistent name of variable in RuleParserSebastien Gonzalve2015-04-241-14/+14
| | | | | | | | Remove misleading hungarian notation of variable name in CRuleParser::iterate() that was making the code hard to read/maintain. Change-Id: I7a034d7999f419263551c3ed9a7c26cda38e3818 Signed-off-by: Sebastien Gonzalve <sebastien.gonzalve@intel.com>
* Type mismatch corrections (Windows 64 bits)Patrick Benavoli2015-04-2441-166/+226
| | | | | | | | This patch removes the type mismatch warnings revealed by Windows 64 compiler. Wherever necessary, used size_t type for size related data. Change-Id: Ie045ce95940cd83fe8d681168ac9526fc6028d43 Signed-off-by: Patrick Benavoli <patrick.benavoli@intel.com>
* CXmlFileIncluderElement constructor cleanupPatrick Benavoli2015-04-241-3/+2
| | | | | | | | Made _bValidateSchemasOnStart attribute initialized as part of the constructor's initializer list. Change-Id: I8f345b5c17f33e012b69005e01dc31a270e41778 Signed-off-by: Patrick Benavoli <patrick.benavoli@intel.com>
* Change blackboard write/readString prototypeSebastien Gonzalve2015-04-243-16/+9
| | | | | | | | | | | | | | Those functions are used in one place and let the caller transform a std::string to a char * which makes the use of alloca necessary. Moreover, the readString cannot know if the pointer given in input points to a buffer with enough space, making it dangerous. This patch reworks the API so that write/readString() get std::string and lets the implementation play with char *. This also allows to get rid of alloca() use which is quite dangerous. Change-Id: I973e4987278289bb914c4e194b86e924a1c88a6d Signed-off-by: Sebastien Gonzalve <sebastien.gonzalve@intel.com>
* Make line endings consistent in schemas filesSebastien Gonzalve2015-04-245-354/+354
| | | | | | | Some xsd files did not have linux style line endings. Change-Id: Id2ec46bf740679e5736bd7283377c610f7c320f9 Signed-off-by: Sebastien Gonzalve <sebastien.gonzalve@intel.com>
* Removal of unnecessary execution modeSebastien Gonzalve2015-04-242-0/+0
| | | | | | | | | Spurious execution mode removed from files: - parameter/Element.cpp - utility/convert.hpp Change-Id: I5d47d505e8c3d9f55ad975c34b51d7caa4905ab9 Signed-off-by: Sebastien Gonzalve <sebastien.gonzalve@intel.com>
* Removed spurious ";" after function definitionSebastien Gonzalve2015-04-241-1/+1
| | | | | | | | CElementBuilder constructor had a ";" right after its defintion. This patch removes it. Change-Id: I60551bf9610b1760181d9b206a98f566cef78f0b Signed-off-by: Sebastien Gonzalve <sebastien.gonzalve@intel.com>
* Remove deprecated project file for Dev-Cpp IDESebastien Gonzalve2015-04-244-448/+0
| | | | | | | File removed from remote-process folder Change-Id: Iec37d2581fd424e7cfd891224813e57a56caa2a6 Signed-off-by: Sebastien Gonzalve <sebastien.gonzalve@intel.com>
* Prevent multiple inclusions of NativeTokenizer headerSebastien Gonzalve2015-04-241-0/+2
| | | | | | | Added #pragma once on top of the file to prevent multiple inclusion. Change-Id: I351ada48e16bf6bfe84b546da5581b1aa6bce03e Signed-off-by: Sebastien Gonzalve <sebastien.gonzalve@intel.com>
* Utility.h multiple inclusions directive harmonizedSebastien Gonzalve2015-04-241-3/+1
| | | | | | | Replace #ifdef and #define wth #pragma once Change-Id: I60fe1e74a7481be38d9b8e44952e533d7ae80857 Signed-off-by: Sebastien Gonzalve <sebastien.gonzalve@intel.com>
* Require PTHREAD using cmake canonic wayKevin Rocard2015-04-241-6/+2
| | | | | | | | | | Pthread was required by included directly findPthread.cmake then testing for success. Use the find_package command and require success with the REQUIRED option. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
* Remove explicit server stopKevin Rocard2015-04-241-3/+0
| | | | | | | | | | | | | | Stopping the remote-processor in the stop command has two side effect: - stopping the remote-processor thread will fail as it is not possible to join a thread to itself. This leads to the leak of all the thread object (destructor not called). - the server will not return the exit command status as it has been stopped during it's execution. Remove the remote-processor stop as it will be called during it's destruction. Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
* [coverage] Correct line number msgKevin Rocard2015-04-241-1/+1
| | | | | | | | | | | | | | | When an error occurs during input parsing, the line number was incorrectly displayed. In python, enumerating an iterable will yield an pair of index and a value. This index starts at 0: next(enumerate([""])) == (0, "") This index was displayed as the line number. Add one to the index to get the line number. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
* [coverage]Quote user value in error msgsKevin Rocard2015-04-241-3/+3
| | | | | | | | | | | | | Some pfw bug induce empty criterion value, when log it is dificult to see the the empty value. Add quotes around values. before: Error in criterion After : Error in criterion "" Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
* functional-tests: remove dumpElement test caseDavid Wagner2015-04-242-136/+0
| | | | | | | This test is failing (on travis only) for strange reasons. As we can not trust it, let's remove it for now. Signed-off-by: David Wagner <david.wagner@intel.com>
* Add functionnal tests to the pfw.Adrien M2015-04-2451-187/+999
| | | | | | | | | | | | | | | | This patch adds functionnal test to the PFW. The execution of these tests needs the test-subsystem to be compiled. The tests can be launch with 'make test'. We set the environment vars in the cmake. Then we launch ACTCampaignEngine.py. Temporary files like build time conf files are placed on the build directory. Next steps : - Test-subsystem needs to be reworked (with binding python). Signed-off-by: Adrien M <adrienx.martin@intel.com>
* Dissociate 'make test' & 'make install'Adrien M2015-04-245-3/+33
| | | | | | | Possibility to run the test without having to lunch the 'make install' command. Signed-off-by: Adrien M <adrienx.martin@intel.com>
* Fix tUINT32_ARRAY index overflow testJules Clero2015-04-241-1/+1
| | | | | | This test was using the wrong array size. It leads to false positive. Signed-off-by: Jules Clero <julesx.clero@intel.com>
* Fix tSTRING_128 parenthesis errorJules Clero2015-04-241-2/+10
| | | | | | | | | This test was trying to set string parameter containing parenthesis through remote-process without escaped them. This patch introduces the escaped value in order to correctly use the remote-process command through bash. Signed-off-by: Jules Clero <julesx.clero@intel.com>