[1.0.1] Linux: Please provide VisionProxy source tree from github as well

Bug #1233062 reported by Gilles Filippini
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
SikuliX
Fix Released
Medium
RaiMan

Bug Description

Hi,

As I understand it, the VisonProxy native lib isn't an external dependency but an internal component of Sikuli. As such, its source tree should be available from github as well, with a proper 1.0.1 tag.

Many thanks in advance.

Revision history for this message
RaiMan (raimund-hocke) wrote :

For now, I can create an additional Git repo SikuliX-Natives, that contains the source tree, as it is contained in the downloadable package
https://launchpad.net/sikuli/sikulix/1.0.1/+download/Sikuli-1.0.1-Supplemental-LinuxVisionProxy.zip
(this contains a template to build VisionProxy.so)

Ok?

summary: - Please provide VisionProxy source tree from github as well
+ [1.0.1] Linux: Please provide VisionProxy source tree from github as
+ well
Changed in sikuli:
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → RaiMan (raimund-hocke)
milestone: none → 1.1.0
Revision history for this message
Gilles Filippini (pini.deb) wrote :

Yes, that would be ok. Please make sure that "__MACOSX" and ".DS_Store" paths are not imported into the github repo.
Thanks.

Revision history for this message
RaiMan (raimund-hocke) wrote :
Changed in sikuli:
status: In Progress → Fix Released
Revision history for this message
Gilles Filippini (pini.deb) wrote :

It's ok but for the sampleBuildVisionProxy script which has errors.

The line:
 includeParm=...
should have its right part surrounded by double quotes because bash doesn't want spaces in assignements:
 includeParm="..."

The $includeParm part of g++ commands should be moved after the filename for the dependencies tracking, because the filename is a parameter of the -MF flag:
 g++ ... -MMD -MP -MF build/<sourcename>.o.d $includeParm ...

Clark Rawlins sent a fixed script on the Sikuli-dev list a couple of hours ago.

Thanks.

Revision history for this message
RaiMan (raimund-hocke) wrote :

Thanks for input.

I updated the repo.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.