Page MenuHomePhabricator

enlightenment_remote: bg-add break
Needs ReviewPublic

Authored by thierry1970 on May 15 2019, 2:35 AM.

Details

Reviewers
raster
devilhorns
Summary

It is no longer possible to change the wallpaper e19 with "enlightenment_remote".
The correction is a simple update of the code.

Diff Detail

Repository
rE core/enlightenment
Branch
arcpatch-D8897
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 11982
Build 8826: arc lint + arc unit
thierry1970 created this revision.May 15 2019, 2:35 AM
thierry1970 requested review of this revision.May 15 2019, 2:35 AM
devilhorns resigned from this revision.May 15 2019, 2:42 AM
devilhorns requested changes to this revision.Jun 26 2019, 11:07 AM
devilhorns added a subscriber: devilhorns.
devilhorns added inline comments.
src/modules/msgbus/msgbus_desktop.c
101

These are all unused variables now so those should be removed from this function (zone, desk_x, desk_y).

130

These are all unused variables now so those should be removed from this function (zone, desk_x, desk_y).

This revision now requires changes to proceed.Jun 26 2019, 11:07 AM

Also, in data/tools/enlightenment_remote .. the help text for desktop-bg-add & desktop-bg-del still show container, zone, desk_x, desk_y as options to be passed in, so likely the help text should be updated also

thierry1970 marked 2 inline comments as done.
  • Removing unused variables (comment: devilhorns).
  • Removing unused variables (comment: devilhorns).
  • Updated help for options: desktop-bg-add and desktop-bg-del.

This looks good now. I will run a test with it tomorrow morning, and if everything is ok then I will push it to git master. Thank You :)

devilhorns resigned from this revision.Jun 27 2019, 6:09 AM

Ok so, I applied this patch and attempted to test it, however the background never changed when I executed the command so I cannot confirm that this patch actually works. I am going to see if @raster can take a look.

devilhorns edited reviewers, added: raster; removed: devilhorns.Jun 27 2019, 6:09 AM
thierry1970 added a comment.EditedJun 27 2019, 7:16 AM

Sorry, I tested with compatibility.
I have a clue error, I correct.

  • Update option index for desktop-bg-add.