Add error logs on revoke_security step

In addition, an error message is printed as an error, not a debug
message.

Change-Id: I1adb4fe8615a33652d3b00875adad52a6c9627d4
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Fix wrong manifest xml file name

The manifest file name is wrong. Desired xml filename is
'org.tizen.hello.xml', but the current implementation removes last
extension and put '.xml' at the end.
This commit fixes the name of the generated manifest xml file.
For wgt packages, '*.wgt.xml' is present, but that's not the issue.

Change-Id: I57e2506898f17c0853fa42f3172225b69cd9819c
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Fix bugs in common steps

* Fixed generating xml routine, that each privilege have its value
  as a data, not a 'name' attribute.
* Changed the iteration routine, from the first item of the privilege
  list.
* Excluded routine removing xml file, from the 'undo' phase of the
  step_record.

Change-Id: Ied4ff4a48ab0a62f93fbb4fac40faa00952eaefe
Signed-Off-By: Youmin Ha <youmin.ha@samsung.com>

Refactor name unpack_directory()

The method unpack_directory() just returns the unpack_directory string,
    but its name implies for class users to "Unpack the directory".
    This commit changes the name from unpack_directory to
    unpacked_dir_path, of the getter, setter, and private variables.

Change-Id: I7259bbb3c8630d7464a98707162d955de45c9432

Give the package type as a AppInstaller object

To send a start/end/progress DBUS signal, client needs to know the
package type as a string. This patch adds an argument package_type on
the AppInstaller constructor, and stores it into the ContextInstaller.

Change-Id: I18a7770dbb706a6ab72cce6010d348fda4d47137

Add parser's handler: appwidget

Change-Id: I3d4009960593e319abc9034e5c6e27b34e064e25

Add parser's handlers: metadata, navigation, splash screen

Change-Id: Idad70b51377d8020a4cc71ae44e382a869ff4a3c

Add setting parser's handler

Change-Id: I2063f0aa5046f5981b2abbb41576d3d1150e9009

Add IME parser's handlers

Change-Id: I8b316d4ce04f05397e05903cd3a921cd05013637

Escape encoded character in URLs in manifest

Change-Id: I30667246ca44bd66ca80953ac9f54555a2564c34

DISALLOW_COPY_AND_ASSIGN(AppInstaller)

Change-Id: I641fa3bf29ec3147e54430259eb6a6d87596c202

Logging unification

Because of existing idea of tagging step in logs,
this commit introduces one more macro:

SCOPE_LOG_TAG(TAG)

Defining this macro in given scope will change log tag of that
scope.

Example:

class StepXYZ : public Step {
  void func() {
    LOG(DEBUG) << "Status is ...";
  }

  SCOPE_LOG_TAG(XYZ)
};

Every log from class StepXYZ will be place with "XYZ" tag.

Change-Id: I8a8e192b0a746dae19a961e238a798b9b2c0c4b9

Remove utf16 handling

This code is not needed.
There is no need for utf16 error codes of parser.

value.h make no longer assumtions on string
encoding - there is no check.

Change-Id: I2d22b8683b27a659c237c0fd1bf167e37c15db70

[Widget-manifest-parser] Added parser's base

Change-Id: Ic7325a08d9ed1359a6e6be77e0d62b3dbe046e7f

[Widget-manifest-parser] Add manifest class and manifest constants

This change adds Manifest class, which is internal representation
of manifest file in widget-manifest-parser.

Change-Id: Icf9a53398fe3941cb19e36d9bacad721ffa0b057

Move logging and macros to utils

Just moves code. I will think about appling this.

Change-Id: I6e6e21a9144f548697acc0f959e7aa2338831121

Merge "Add utils library." into tizen

Send pkgmgr signal always

Zip or config.xml can be malformed.
This fixes handling signalling in those cases.

Code which handles pkgmgr signal was extracted into
 - src/common/pkgmgr_signal.*

Change-Id: I46b96f6ba8cd2c41110f2fefbceb6dc1b2969216

Merge "Add url decoding for 'uri' attribute of signatures" into tizen

Add url decoding for 'uri' attribute of signatures

The value of URI attribute of signature files can be encoded uri.
But, SignatureValidator can't recongize encoded 'uri'.
Add url decoding routine to SignatureParser.

Change-Id: Ifec7b04b2a591bddca15fbe3cdafa90d2a0964c5

Add utils library.

It is supposed to contain common utilities, which can be used by
other parts of code. This commit contains utf_converter code,
which allows convertions between std::u16string and std::string.
There is also DictionaryValue class added, which is needed for
saving nodes of widget manifest.

Change-Id: Ibe4265e6dc3a7f78637d0103222d87d16b70bc2a

Remove double installer creation

Change-Id: Id8842d3dcb471a00437f3d620229a6c45a97989f

Manifest generation fix

xmlTextWriterEndDocument closes all not closed elements (nodes).
Therefore it faked that manifest is ok.
Some elements were not closed properly.

Change-Id: I0b4e1a0051c25dff10a67a7ac8809295d44d5c5b

Fix Name field in case of widget label

If the short name is empty use the long name instead

Change-Id: Id7f6bd3fa54a9801c0f93e510ae6f723241185e4
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Fix Description field in Tizen Manifest

This permits to avoid to have this

<label>(null)</label> instead of <label></label> in the manifest

Change-Id: Iddaedf33940358cd52590487f6a7c6febda809c1
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Fix Application type in Generate XML Manifest step.

The application should depends of the original type set in the original manifest.

Now, use the one provided by step parser

Change-Id: I6aa34cf1dc2ebc9733f6239125aa12f441de47c0
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Add new specific step to create symbolic link for WebApp

Remove this operation from generate xml step, create a new one

Add a spec file option for set the WRT binary path used for symbolic link

Change-Id: Ife93a10b830fd093dc9c02184dd2490ce99d67a0
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Add manifest to wgt-backend package.

This is needed to assign the "User" smack label to wgt-backend

Change-Id: Icea7b182c89a7e8ff631f6f6ce9f188d180c8fd7
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

StepSecurity

Adds security manager API calls to apply correct security context
during installation and uninstallation process.

Change-Id: Ieceb1126edfe65f1f1f438e9c4dea7907e9f3e27

Step class design issues

 - lack of virtual destructor
   It is obligatory in classes that are extended.
   This case may lead to memory leak or undefined behaviour.

 - unnecessary argument passing to each function,
   We have 'class' to keep information within.

 - memory management in AppInstaller class of allocated Steps,
   Most reliable way is to use smart pointers.
   Use new/delete only if it is really neccesary.

 - fixes errorcodes to some understandable ones,
   Any function should not return int that cannot be interpreted
   what it actually represents.

 - catching int,
   There was some mistake (?). Use std::exception derived classes in
   exception handling if needed.

Change-Id: Iba8ea02dde503ec1b2f043ed5965a537d6a58f51

Fix error Management

If Undo method fails the Error's signal is not sent to pkgmgr-server
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>
Change-Id: Ib867d1a0f488787b2408f914e66ab561d225d65a

Add Send signal Step

Change-Id: I3cf119442f2d231ebd4aab84790e2af12c792182
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Add manifest for wgt-backend

Change-Id: I8221bed3fd57e653745293d82df6ab181fddf8f6
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Assign pkgid in installation context

Fixes installation process

Change-Id: I7cc7ae9d8f7e09c98fb5eb007abb33dd1c194beb

Merge "Get rid of cmake's CMP0023 warning" into tizen

Adding Uninstallation features

Adding Methods to get Root Applications path and Application path
Adding Step remove to remove files of 1 package
Adding Step to unregister a package

Signed-off-by: Sabera Djelti (sdi2) <sabera.djelti@open.eurogiciel.org>
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>
Signed-off-by: Pawel Sikorski <p.sikorski@samsung.com>
Change-Id: I3d7bd76a75b413e310b0804b3856b021d4115e13

Add StepParse::fillManifest method

The purpose of this method is to fill the Context structure with the information
retreived from config.xml file

Change-Id: Ia0777a0251d967611a4b332337d651cfee3ef830
Signed-off-by: Pawel Sikorski <p.sikorski@samsung.com>
Signed-off-by: Sabera Djelti (sdi2) <sabera.djelti@open.eurogiciel.org>
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Fix Manifest generation step

-> Align Privileges section generation with the specs
-> Add robustness when there is neither svc app information nor ui app information

Change-Id: I76de58d65a67b508c7777d0b964c00a4b74fc501
Signed-off-by: Sabera Djelti (sdi2) <sabera.djelti@open.eurogiciel.org>
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

add step_generate_xml & step_record

step_generate_xml creates the manifest pkgid.xml
step_record stores app data in database and creates desktop file

Change-Id: Ie7920417c71f17fbf6deff19935d9d95b7debfa0
Signed-off-by: Sabera Djelti (sdi2) <sabera.djelti@open.eurogiciel.org>

Get rid of cmake's CMP0023 warning

http://www.cmake.org/cmake/help/v3.0/policy/CMP0023.html

Change-Id: Icb3bd90104b50b9f498f9e9911d479a67e682c72

Add step-parse

- compile successfully
- widget-manifest-parser library added
- widget-manifest-parser library works correct
- storing data from widget-manifest-parser proposed

Change-Id: I317554abbf19c72c6c1a77b28218c7008d58cb44

[Cpplint] Stylecheck error - other

Fixes the rest of errors (about 10)

Change-Id: Icfd9147e6615154a688cd7e773b2d679865cf073

[Cpplint] Spacing comments in code

Rules:
 - [whitespace/comments]

Change-Id: I9eb4c1683d7077989b7c50a459596fd3598f8ca1

[Cpplint] Line length limit - 80 characters

Rules:
 - [whitespace/line_length]

Change-Id: I87f5bcad992628250bcd5b92d736379d08159ac8

[Cpplint] Fix whitespaces

Rules:
 - [whitespace/tab]
 - [whitespace/blank_line]

Change-Id: I14868ec04e0416bea8d98b4c4604439c8484d245

[Cpplint] Fix header guards

Rules:
 - [build/header_guard]

Change-Id: I4160f4899fd00cf64fa1cdd6e4ace137b1b256f4

[Cpplint] Single source tree

cpplint.py requires consistent single source code tree.
Otherwise some rules may not apply.

Running:
cpplint.py --root-dir=src/

Will obligate each header file to have correct guard macro.
This will be fixed in future.

===========================================================
To run stylecheck type:
./tools/check-coding-style
===========================================================

Change-Id: I05b62b09ce789648a11a305279adcf8ea5a70d51

add step-copy

step-copy aims to copy the already extracted applications temporary
folder to a <pkg_id>/<app_id> folder.

CopyDir() function based on boost::filesystem interface is added
to utils namespace.

Change-Id: Ibaf32a38eabe92202b9373814968ff97a7e693f7
Signed-off-by: Sabera Djelti (sdi2) <sabera.djelti@open.eurogiciel.org>
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

namespaces reorganization

* common_installer {} - this namespace includes all app-installers definitions
  (Step,  ContextInstaller, utils, AppInstaller)

* utils {} - this namespace includes all extra definitions that are
  not directly related to app-installer framework and are used within
  the project (eg. logging, "dirs" handling, design patterns primitives
  (eg. singleton) etc)

* common/unzip {} - this is a namespace that includes unzipping step
  and its additional definitions

* common/signature {} - this is a namespace that includes signature step
  and its additional definitions

* common/copy {}, common/register {}, etc - these namespaces are
  analogic to unzip or signature

* wgt {} - this is a namespace that includes definitions specified for
  wgt type installation

* wgt/parse {} - this is a namespace that includes parsing step
  specified for wgt type installation

Change-Id: I9fbc542e9184def7bbd6955837c506ccf32825e3
Signed-off-by: Pawel Sikorski <p.sikorski@samsung.com>
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

fix inconsistent naming rule for accessor and build warning

1. get_manifest_data -> manifest_data (context_installer)
2. comment out unused parameter (step_unzip)

Change-Id: Ie8732a759fe888002f7adb792ac6b01f91a09b11
Signed-off-by: Wonguk Jeong <wonguk.jeong@samsung.com>

fix inconsistency of Step related class/file names

as is: StepUnzip(step_unzip.cc), SignatureStep(signature_step.cc)
to be: StepUnzip(step_unzip.cc), StepSignature(step_signature.cc)

Change-Id: I920056464d1e359a615b158e41477c56b56d9b0e
Signed-off-by: Wonguk Jeong <wonguk.jeong@samsung.com>

fill uid in constructor of context installer

Change-Id: Icfe598fea96b4d505ff86f896784acf398a5545e
Signed-off-by: Wonguk Jeong <wonguk.jeong@samsung.com>

source files reorganization

* move step.h and step-unzip.h into ./include/step/ dir
* create utils source files in order to offer some filesystem functions
* fix some coding style errrors

Change-Id: I339c94774f6d1b0e281425f290e3ef49a44ba66d
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

fix dir scanning bug during step unzip

Scan iterator has been stopped on directory.
"unzGoToNextFile" was never invoked in case of directory file.

Change-Id: I66cf31eb7fa56720d1be7b0b56072780e1c2f542
Signed-off-by: Wonguk Jeong <wonguk.jeong@samsung.com>

rework step unzip

- add GetApplicationPath() method to ContextInstaller class
    in order to get user application path.
    This change involves to add context_installer.cc source file
    and add tzplatform-config dependency.
- define 'utils' namespace
- mainly use boost::filesystem interface
- no coding style errors in step_unzip files

Change-Id: Ibae9c9e48ab44793b83d7a55e6db776472fb4610
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

fix coding style in few files

  * app_installer.h
  * app_installer.cc
  * step.h
  * wgt_backend.cc

Change-Id: Ib160a18be703a04e022c4a55a115d3165845fcf3
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

move Context_installer struct to ContextInstaller class

Also fix coding style in context_installer.h

Change-Id: I05522ab8a66fb63d34dc93c156f5bb4a3ebedbfd
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

Signature implementation moved from xwalk

This is digital signature implementation moved from xwalk.
It is based on following PRs:
 - https://github.com/crosswalk-project/crosswalk/pull/2169
 - https://github.com/crosswalk-project/crosswalk/pull/2291
 - https://github.com/crosswalk-project/crosswalk/pull/2422

There are some not solved issues:
 - test correctness for valid widgets,
 - seperation from managing root certificate,
 - style of this code.

[[ PLEASE NOTE THE COMMIT CONTAINS CROSSWALK'S BSD-STYLE LICENSE ]]

Change-Id: I42db7e8a02b44a88ca88d143ad034fd1e253f3e2
Signed-off-by: Pawel Sikorski <p.sikorski@samsung.com>

Adapt check coding style tool to file extension (*.h and *.cc)

Change-Id: I1bcbfc3693018106e42bc8a92423baba9c71f3c9
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

Inotroducing std::string for pkgid, file_name, unpack_directory of
InstallerContext.
Thanks to that these fields are automatically alloacted/deallocated
(no need to call free in case of errors, in case of clean/undo steps, etc)

Additionally, "const char *" instead of "char *" in
* extractTo_u,
* unzip_u

Change-Id: Id16cbf1c7ece36024e197c07f6e05a149d8f1324
Signed-off-by: Pawel Sikorski <p.sikorski@samsung.com>

Using C++ includes instead of C:
* stdio.h --> cstdio
* stdlib.h --> cstdlib
* errno.h --> cerrno
* assert.h --> cassert
* string.h --> cstring

Change-Id: Ie51b397fb8ba6a943b8a279e80afb8cd67a297d7

Style check rules fix

Added:
-build/c++11 - disable errors of using c++11

Removed:
-runtime/references - this rules was not relevant to code

To run style, type:

./tools/check-coding-style

Change-Id: Ie367dd5bfd44c5ea8bb64b4640e443212907863a

Change rpm group classification

Change-Id: Ic6262f806785b72c88b396e20923ccc2dc8a5d9c

fix packaging to improve rpmlint score

- rename spec file
- adapt license syntax
- adapt group tag
- fix %post/%postun scriptlets

Change-Id: Ia692175d8f66fd7046caf6164b91b455e64b8961
Signed-off-by: Corentin Lecouvey <corentin.lecouvey@open.eurogiciel.org>

File name and their extension changes:
* .hxx --> .h
* .cxx -> .cc
* dash ("-") in filenames --> underline ("_")

CmakeLists.txt modified accordingly

Change-Id: I3e61156428971f535608a0d16f58ef6eb5f3536b

Dependencies fix

I made mistake in CMakeLists.txt.
Please use ApplyPkgConfig for any new dependency
as it is done for existing ones.
This reduces boilerplate lines about include_directories
and link_libraries added over and over again.

Change-Id: I25144ea4c6516565e998cb3d71048ad2fe87cf81

Stylecheck script

Change-Id: Ife054126780fc0b6e8fcb3d413f1a44ea71bc4db

add step-unzip

Change-Id: I06aca1845aa1a63c70fdd22ec45bea9a77f6d5bb
Signed-off-by: Sabera Djelti (sdi2) <sabera.djelti@open.eurogiciel.org>

Fix & improve cmake

Change-Id: Icee2e5cfbd8afa9f067d0fad11a1f386910949ad

Add Skeleton and basis class

Change-Id: I90aeaef9904c173ea5cd76d38ba874e9f5e239fc
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Initial empty repository