summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2015-10-06android/python bindings: force gcc and 32bits compilationDavid Wagner
Since Android only provides a 32bits version of the python interpreter and libs, we need to compile for 32bits even on 64bits hosts. Also, since it seems that clang is the default compiler for host, we need to explicitely forbid it as long as the parameter-framework is compiled with gcc. Change-Id: I5d484c0e58c304680192f443acd81258de179e52 Signed-off-by: David Wagner <david.wagner@intel.com>
2015-08-27Disable Werror for unused-result.Dan Albert
Nothing in this project checks results of the C library calls. This should definitely be fixed, but we'll want to fix it upstream and want to unblock the build for now. Change-Id: Ia7bcca9022e9f5a5e3fa44e67b3fe41673cf2de3
2015-08-26stlport library is needed for versions of Android before MZhengyin Qian
Conditionally include it based on the Android version Change-Id: I4b17e3c0f61a6fd2b0b1e48ed1efe16e0a446dfc
2015-08-26Fix build from stlport.Dan Albert
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)
2015-02-19Merge pull request #51 from cc6565/fix-pipe-issueDavid Wagner
Output parameter within getNumericalValue not initialized
2015-02-19Output parameter within getNumericalValue not initializedFrancois Gaffie
When calling getNumericalValue on inclusive criterion, the value to be outputed is never initialized. The Criterion type will append the value corresponding to each literal value to the uninitialized value leading to undefined behavior. This patch initializes the output value. Issue: GMINL-6007 Change-Id: Id9fc8b010b4d0b6dfc921825387817789d35e5ba Signed-off-by: Francois Gaffie <francois.gaffie@intel.com>
2015-02-19Merge pull request #50 from dawagner/make_parametermgr_init_privateDavid Wagner
Make CParameterMgr::init private
2015-02-18Make CParameterMgr::init privateFrederic Boisnard
Since 3adb785eb097028750fc1b4c5ecab3bebf1a9ae3, CParameterMgr::init should not be called directly. However, it was kept public and CParameterMgrFullConnector was using it. Make it private and remove its use from CParameterMgrFullConnector. Signed-off-by: Frederic Boisnard <frederic.boisnard@intel.com> Signed-off-by: David Wagner <david.wagner@intel.com>
2015-02-18Merge pull request #43 from dawagner/fix-systemclass-toxmlDavid Wagner
Fix SystemClass::toXml and remove useless overriden methods
2015-02-18SystemClass: remove useless overridden init() methodDavid Wagner
The overridden implementation was only calling the base implementation. It can therefore can be completely removed as it is the default behaviour. Change-Id: Ic0eadf82abfdaad0ca76d3e9a757544d5ec31427 Signed-off-by: David Wagner <david.wagner@intel.com>
2015-02-18Merge pull request #41 from krocard/fix_string+num_concatDavid Wagner
Fix pointer arithmetic instead of concat
2015-02-18Merge pull request #49 from pafonso/iminleg_rebaseDavid Wagner
Iminleg rebase
2015-02-18Fix SystemClass duplicate 'Name' attribute in xml exportDavid Wagner
When refactoring Element::toXml, we forgot to make the necessary adaptation in SystemClass::toXml. The latter does not need to set the Name attribute anymore because this is taken care of by the base toXml method. SystemClass::toXml now only calls the base and can therefore be completely removed. Change-Id: Ibd12feaac2c101dcc22e8716e332cb02f03ba99a Tracked-On: https://jira01.devtools.intel.com/browse/IMINAN-19676 Signed-off-by: David Wagner <david.wagner@intel.com>
2015-02-18Add libremote-processor as dependencyPhilippe Afonso
libremote-processor headers are needed by the parameter core. This patch adds explicitily the dependency to this module. Change-Id: I7a223a0abd35f9524a047ca92e1029c5cef38e38 Tracked-On: https://jira01.devtools.intel.com/browse/IMINAN-19676 Signed-off-by: Philippe Afonso <philippex.afonso@intel.com>
2015-02-18Create output host library directory if neededPhilippe Afonso
The directory where to generate output may not exists leading generation to failure. This patch adds the directory creation and thus makes sure generation won't fail for this reason. Change-Id: I1cee75d6d7921315dc3a928245499bde447c3639 Tracked-On: https://jira01.devtools.intel.com/browse/IMINAN-19676 Signed-off-by: Sebastien Gonzalve <sebastien.gonzalve@intel.com>
2015-02-18Create a Logarithm Adaptation parameter typePiotr Diop
Some parameters need to be converted to logarithm. This can be the case for volumes that come from HAL as mere decimal values. This patch adds the LogarithmAdaptationParameter type. Change-Id: Ic64232816a82c56a6ddffa5fdccda356eb4ec9c7 Tracked-On: https://jira01.devtools.intel.com/browse/IMINAN-16954 Signed-off-by: Piotr Diop <piotrx.diop@intel.com> Reviewed-on: https://android.intel.com:443/320371
2015-02-18[Workaround][xmlGenerator] Increase retry on EADDRINUSEKevin Rocard
During host domain generation, a port must be chosen before spawning a servers, producing an error if someone else uses it before the server. This race was workaround by retiring 10 times before failing. This divided the failure probability by 10, but is no longer sufficient. A correct implementation without this race is currently been developed (without socket) but until then change the retry number from 10 to 1000. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com> Tracked-On: https://jira01.devtools.intel.com/browse/IMINAN-17358 Change-Id: Ie80e4508952ef150eb999d9a6fa7e60b61825f35 Reviewed-on: https://android.intel.com:443/319856
2015-02-18Remove remote process socket connection timeoutKevin Rocard
From time to time, on Buildbot, the PFW generation build step fails due to remote processor socket connection timeout. In order to avoid deconnection when the peer is slow, this patch removes the remote processor socket connection timeout. Change-Id: I56b5b4b846a7489accd56ba3a3224dc1ffc1085e Tracked-On: https://jira01.devtools.intel.com/browse/IMINAN-5511 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: https://android.intel.com:443/289012
2015-02-18Cleaned up XML export featurePatrick Benavoli
BZ: 209937 This patch solves 1 issue regarding XML export feature: - ParameterMgr.cpp::exportElementToXMLString(): This method was receiving a useless bValidateWithSchema argument. Indeed, elements in memory are necessarily valid (unlike documents coming from external sources). Change-Id: I55bf84c8ff53c37c9fc55e60f7f64e8776a43dc7 Signed-off-by: Patrick Benavoli <patrick.benavoli@intel.com>
2015-02-18Missing error reporting statement in XML Export featurePatrick Benavoli
BZ: 209937 In case an error is encountered during the XML encoding operation, no error message is created to log the issue. This patch updates XmlStringDocSink.cpp::doProcess() so that the error string is filled in case of errors. Change-Id: Ie93e50f857506b8637b1bb0dd8d411a01f16ec49 Signed-off-by: Patrick Benavoli <patrick.benavoli@intel.com>
2015-02-18Adding XML format to Criteria export tuning I/FPatrick Benavoli
BZ: 209937 The "listCriteria" command returns the list of criteria in a human readable way. However, external tools that require to know this list of criteria have trouble parsing this output. An XML output would solve the issue. Changed "listCriteria" command to accept "XML" as first and only argument. In the end, listCriteria command accepts the 3 following forms: - listCriteria => will list the criteria states and type content in a human readable format - listCriteria csv|CSV => will list the criteria states and type content in a CSV format - listCriteria xml|XML => will list the criteria states and type content in an XML format Removed python binding accordingly. Change-Id: Ib060ec0a5d1ff87ba6c25caf4e0d5839a7927715 Signed-off-by: Patrick Benavoli <patrick.benavoli@intel.com> Signed-off-by: Sebastien Gonzalve <sebastien.gonzalve@intel.com>
2015-02-18Delete subsystem instances *before* plugin unloadPatrick Benavoli
BZ: 208504 Before this patch there was a crash in CElement destructor in Ubuntu upon destuction of parameter framework instance. The crash was caused by the attempt to call unmapped code: The subsystem plugin libraries were getting unloaded before the execution of the subsystem destructors. Note: In Android the issue is not seen since library unload is not implemented. This patch ensures SystemClass destructor destroys the subsystems explicitely unloading the plugins. Change-Id: I19dad262b384bdbd63c7c319a41a5d547d0e75e9 Signed-off-by: Patrick Benavoli <patrick.benavoli@intel.com>
2015-02-18Coding style correctionPatrick Benavoli
BZ: 208504 Concerns the handle to remote processor library. In PFW coding style, the variables are named like in english language: The actual nature of the object needs to lie at the end of the name. Change-Id: If41218c3456d07c81a953182488c6ae71e31b6d8 Signed-off-by: Patrick Benavoli <patrick.benavoli@intel.com>
2015-02-18Removed spurious execution mode on source filesPatrick Benavoli
BZ: 208504 Some source files were mistakenly assigned the executable flag. Change-Id: Iad85123c331d2599126ed46f070fe10614f7c360 Signed-off-by: Patrick Benavoli <patrick.benavoli@intel.com>
2015-02-17Merge pull request #40 from krocard/no_verbose_cxxDavid Wagner
Remove CXX compiler verbose flag for swig binding
2015-02-17Merge pull request #48 from krocard/remote-process_completionDavid Wagner
Introduce remote-process bash completion
2015-02-17Introduce remote-process bash completionKevin Rocard
When working with the parameter-framework, the remote-process command is very handy. Nevertheless it has long commands and it is easy to make mistakes. Add a bash completion script to complete: - hostnames - frequently used port - commands and arguments based on the remote usage (help command) Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
2015-02-16Low case cmake command in bindings for consistencyKevin Rocard
Some cmake commands were in big case in the python binding cmake. As all others are in low case, put everything in low case. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
2015-02-16Remove CXX compiler verbose flag for swig bindingKevin Rocard
When compiling swig generated files, the CXX compiler flag list included -v. This leaded to unnecessary log output during compilation. Remove the flag Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
2015-02-16Fix cmake set_property in python bindingsKevin Rocard
`SET_SOURCE_FILES_PROPERTIES(pfw.i PROPERTIES SWIG_FLAGS "-v" "-Wall" "-Werror")` sets 2 variables: - `SWIG_FLAGS="-v"` - `-Wall="-Werror"` which is obviously not what was wanted. Replace by `SET_PROPERTY(SOURCE pfw.i PROPERTY SWIG_FLAGS "-v" "-Wall" "-Werror")` that only sets `SWIG_FLAGS="-v -Wall -Werror"`. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
2015-02-16Fix pointer arithmetic instead of concatKevin Rocard
`"my number " + 5` does not produces `"my number 5"` but rather `"mber "`. If the number was greater than `len(string) + 2` (pointing to the \0 and one past the end in authorized, more in UB) the result would have been undefined behaviour. Transforme the number to std::string using toString. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
2015-02-13Merge pull request #47 from dawagner/krocard-missing_std_in_skeletonDavid Wagner
missing std in skeleton
2015-02-13Travis-CI: also build the skeleton pluginDavid Wagner
The skeleton plugin is an example and as such needs to work out-of-the-box. Make travis build it to ensure we don't break anything. Signed-off-by: David Wagner <david.wagner@intel.com>
2015-02-13Merge branch 'missing_std_in_skeleton' of ↵David Wagner
https://github.com/krocard/parameter-framework into krocard-missing_std_in_skeleton
2015-02-13Merge pull request #45 from krocard/xml_coverage_create_elementDavid Wagner
Use minidom factory methods in xml coverage generator
2015-02-13Merge pull request #46 from 01org/travis-officialDavid Wagner
Add a configuration file for Travis-CI
2015-02-13Add a configuration file for Travis-CIDavid Wagner
Travis will build each new commit and each pull request and publish the results here: https://travis-ci.org/01org/parameter-framework and on the #parameter-framework channel on FreeNode. This commit also adds a build status icon in the Readme. Signed-off-by: David Wagner <david.wagner@intel.com>
2015-02-10Use minidom factory methods in xml coverage generatorKevin Rocard
The exportToXML was creating an minidom.Element using it's constructor, which is forbidden in the documentation: > Applications should not instantiate the classes themselves; they should use the creator functions available on the Document object. http://docs.python.org/library/xml.dom.minidom.html Since python 3.4 this method no longer works (private interface change) thus the xml coverage generation was broken. Fix by calling the recommended method Document.createElement. See bug: http://bugs.python.org/issue15290 Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
2015-02-06Missing std prefix in skeleton pluginKevin Rocard
The skeleton plugin is not compiled when compiling the root pfw project. As a result, when `using namespaces` was removed from the pfw headers the skeleton plugin was left unchecked and unfortunately broken. Fix the missing `std::` by adding them where needed or inserting a `using std::string`. Signed-off-by: Kevin Rocard <kevin.rocard@intel.com>
2015-01-29Merge pull request #21 from ↵David Wagner
krocard/Fix_error_message_in_case_of_unknown_criterion_value Fix error message in case of unknown criterion value
2015-01-29Adding a missing const in a get type methodPatrick Benavoli
BZ: 208504 CParameterAccessContext::outRawFormatIsHex() was missing the const modifier. Change-Id: I3120a231d2dff8f4274b95cd2d3c8ff0817bf8ac Signed-off-by: Patrick Benavoli <patrick.benavoli@intel.com>
2015-01-29Move element tree initialisation to load functionGuillaume Denneulin
BZ: 207091 The parameter platform manager used to initialize the element tree. However, this is too early for plugins, since they need information from the core which have not been initialized yet. This patch delays the initialization of the element tree by adding it to the load() function of the ParameterMgr. Change-Id: I98e0e3a20ac0af2736eddc4c8349911eb1d1a691 Signed-off-by: Guillaume Denneulin <guillaume.denneulin@intel.com> Signed-off-by: Mattijs Korpershoek <mattijsx.korpershoek@intel.com> Reviewed-on: https://android.intel.com/215426 Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Reviewed-by: Benavoli, Patrick <patrick.benavoli@intel.com> Reviewed-by: Rocard, KevinX <kevinx.rocard@intel.com> Tested-by: Rocard, KevinX <kevinx.rocard@intel.com>
2015-01-29Fixing TestPlatform.cpp file coding styleJules Clero
BZ: 213233 TestPlatform.cpp has been edited regarding uncrustify suggestions. Change-Id: Iac75c5923d5bf9836730ca6f21789967a98895e9 Signed-off-by: Jules Clero <jules.clero@intel.com>
2015-01-29Fix test-platform set criterion by Lexical State BugJules Clero
BZ: 213233 The setCriterionState command of the Test Platform was setting undesired value when asking for multiple states to an InclusiveCriterion. The method used in setCriterionStateByLexicalSpace was to parse arguments before to get numerical value for each of them with the getNumericalValue method from ISelectionCriterionTypeInterface. Then, different masks were aggregated to set the criterion's state. Regarding the getNumericalValue method behaviour which parses automatically value arguments, this method leads to undesired bytes in the mask which leads to undesired criterion's state. The solution is to aggregate the setCriterionState arguments when more than one arg are used. As getNumericalValue use | to split his first parameter, the code has been modified to use this behaviour. getNumericalValue is now called only one time and the returned mask is directly used to set criterion's state. Change-Id: Ie3a1d79a6f66928a2418f11233ff0d1ef5c3e900 Signed-off-by: Jules Clero <jules.clero@intel.com>
2015-01-29Merge pull request #38 from dawagner/python-bindingsDavid Wagner
Python bindings and new EDD -> XML generator using these bindings
2015-01-28python bindings: add a READMEDavid Wagner
Change-Id: I361ef7ca42fb79f61164f227b8f7893f90332b62 Signed-off-by: David Wagner <david.wagner@intel.com>
2015-01-28xmlGenerator: update the READMEDavid Wagner
Document the new python xml generator and deprecate the old one. Change-Id: I5bf9a8206c82678eda7b29b1fe88cbc0396b1f9d Signed-off-by: David Wagner <david.wagner@intel.com>
2015-01-28domainGenerator.py: disable the remote interfaceDavid Wagner
The remote interface sometimes gets in the way of XML generation when the port is already in use. A "retry" mechanism existed in the previous generator but was unreliable. In the new generator, we disable it altogether as we don't need it. Change-Id: I0a1850058b688f5ea09b93959601d3c706d812b6 Signed-off-by: David Wagner <david.wagner@intel.com>
2015-01-28ParameterMgr: add a method to forcefully disable the remote interfaceDavid Wagner
In some setup, users might not want to start the remote interface even if the toplevel configuration file allows it. The parameter-framework client can now override the remote interface starting policy. This was, until now, only dictated by the 'TuningAllowed' attribute in the toplevel configuration file and the presence of the libremote-processor library. This method is forwarded to the connectors and the bindings. Change-Id: Ib6dc272dfc7114125fdafd1a58642cde88847752 Signed-off-by: David Wagner <david.wagner@intel.com>
2015-01-28domainGenerator.py: pass a logger to the parameter-frameworkDavid Wagner
Take advantage of the setLogger() method that was newly added to the bindings to allow the parameter-framework to log during XML generation. Change-Id: I71a43fae915756a7986a48a840306d454d071f17 Signed-off-by: David Wagner <david.wagner@intel.com>
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-18 12:22:09 +0000