Add support for some GENIVI IVI components to AGL

Description

  1. Owner name(s): userID, andrunko

  2. Project maintainers: userID, andrunko

  3. Project name: AGL_GENIVI

  4. Add support for some GENIVI IVI components to AGL (namely, GENIVI Audio Manager, GENIVI Diagnostic Log and Trace - DLT, updated Wayland-IVI-Extension and the corresponding deps)

  5. GENIVI

Environment

None

Activity

Show:

Jan-Simon Moeller April 11, 2017 at 11:34 AM

Is this still needed or actively developed ?

Andre Magalhaes May 5, 2016 at 2:42 AM

I pushed an updated patch for AudioManager to bump the version to 7.3 (I used this version as this is the latest one that has a corresponding AudioManagerPlugins version). I also pushed AudioManagerPlugins for review and you can find more details on https://lf-automotivelinux.atlassian.net/browse/SPEC-122#icft=SPEC-122.

I will be working on the current review comments next.

Jan-Simon Moeller May 3, 2016 at 9:24 PM

So, we managed to get 2 things done:
a) use git review to upload the changes
b) find the above bug in repo where it assumes that username for the gerrit login and username part of the email are equal ... sigh.

Andre can use git review for now. But the repo bug needs to be tracked in a new bug.

Jan-Simon Moeller May 3, 2016 at 9:22 PM

found the issue
url = branch.remote.ReviewUrl(self.UserEmail)
print(self.UserEmail)
print(url)

Andre Magalhaes May 3, 2016 at 8:56 PM

Thanks for the help. We managed to make it work using "git review" instead of repo and the changes are now pushed to meta-agl (still pending the AudioManagerPlugins).

Using repo we still could not make it work as it insists on using the wrong username when doing repo upload.

Here is the log using "repo --trace upload":

... : git var GIT_COMMITTER_IDENT 1>| 2>| : git push --receive-pack=gerrit receive-pack ssh://andre.magalhaes@gerrit.automotivelinux.org:29418/AGL/meta-agl-demo refs/heads/test:refs/for/master 1>| 2>| Permission denied (publickey). fatal: Could not read from remote repository. ...

As you can see it tries to push using "andre.magalhaes" as username, although repo/git is configured to use andrunko as username.

While debugging I saw that on .repo/repo/project.py it gets the committer username using the user email on the method UploadForReview (search for "url = branch.remote.ReviewUrl(self.UserEmail)"). I am not sure if this is on purpose but as my username and email differ it doesnt work with repo unless I change this code or my email .

Won't Fix

Details

Assignee

Reporter

Priority

Created April 15, 2016 at 5:11 PM
Updated June 16, 2017 at 3:52 PM
Resolved June 16, 2017 at 3:52 PM