profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/lucor/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

lucor/fyne-cross 70

Cross Compile Fyne Applications

fyne-io/fyne-cross 60

Cross compiler tool for Fyne apps

lucor/fyne-opensnitch 34

An OpenSnitch UI in Go using Fyne

lucor/easeljs-textarc 8

EaselJs extension allows Text Arc drawing

lucor/docpad-plugin-navlinks 7

Docpad plugin which adds the ability to generate a navbar for documents with links to the next and previous document of the specified collection

lucor/docker-php7-cli 5

Lightweight Docker Image for PHP7 CLI

lucor/docker-ab 3

A lightweight Docker image for ApacheBench based on Alpine linux

lucor/docker-bats 3

Lightweight Bash Automated Testing System Docker Image

lucor/docker-php-composer 3

Minimal PHP Composer Docker Image

lucor/jquery-svg-fallback 2

A jQuery plugin for swapping out svg images with png fallback images when SVG is not supported by browser

Pull request review commentfyne-io/fyne-cross

Support specifying target architectures for Android

 const ( 	ArchArm64 Architecture = "arm64" 	// ArchArm represents the arm architecture 	ArchArm Architecture = "arm"+	// ArchMultiple represents the universal architecture used by some OS to+	// identify a binary that supports multiple architectures (fat binary)+	ArchMultiple Architecture = "multiple"

it was the initial proposal, but we changed to multiple architecture to align with the android docs as pointed out by Andrew. Probably if we want to uniform to the "universal" term across android and iOS / macOS this is a good time :)

lucor

comment created time in 20 hours

PullRequestReviewEvent

pull request commentfyne-io/fyne-cross

[base-image] update Go to v1.16.8 and Fyne CLI tool to v2.1.0

Just a thought form my side. Go 1.17.1 is released and I guess .2 will be in a few weeks. Should it perhaps be time to update to Go 1.17 now?

Agreed :) The idea is to have a docker release with 1.16.x and then release one with 1.17.x so users can use both tags if needed. Thoughts?

lucor

comment created time in 20 hours

push eventfyne-io/fyne-cross

Jacob

commit sha 1078603deea1064d2f3a3c26c37b706811bf2ffe

Fix the link to download macOS tools The old one does not work any more. This fixes it and makes the search term prepared for the user.

view details

Jacob

commit sha 8193427d4c13f14e247806246074b4e7f34bdcf2

Merge pull request #61 from Jacalz/fix-macos-ink Fix link to download macOS tools

view details

Andy Williams

commit sha a002416c392157711521a80cf1041b2918012166

Let go get upgrade

view details

lucor

commit sha 33ab4155c0f96cc3f35f26ab9b83b66f4b4c0120

Merge branch 'master' into base-image-update

view details

lucor

commit sha 5124b7ab01aaa25f3afa577b8d165d1353be3f7b

This commit updates the docker base image. - Update Go to v1.16.8 - Update fyne CLI to v2.1.0

view details

Luca Corbo

commit sha 80ebac0db5f342e7841595343079863324a9abed

Merge pull request #67 from lucor/base-image-update [base-image] update Go to v1.16.8 and Fyne CLI tool to v2.1.0

view details

push time in 20 hours

PR merged fyne-io/fyne-cross

[base-image] update Go to v1.16.8 and Fyne CLI tool to v2.1.0

This PS updates the docker base image with:

  • Go to v1.16.8
  • fyne CLI to v2.1.0
+8 -4

0 comment

3 changed files

lucor

pr closed time in 20 hours

push eventlucor/fyne-io-fyne-cross

lucor

commit sha f318e6e67e36b181a1893385d9e6ced7b8764d95

Update to use multiple architectures in place of universal

view details

push time in 5 days

PR opened fyne-io/fyne-cross

[base-image] update Go to v1.16.8 and Fyne CLI tool to v2.1.0

This PS updates the docker base image with:

  • Go to v1.16.8
  • fyne CLI to v2.1.0
+40 -28

0 comment

10 changed files

pr created time in 5 days

create barnchlucor/fyne-io-fyne-cross

branch : base-image-update

created branch time in 5 days

issue commentfyne-io/fyne-cross

Compile to andriod error

It was my understanding that the code provided above was working correctly, hence my question. @xfengw could you please confirm and in case share the code along with the steps to reproduce the issue ?

xfengw

comment created time in 6 days

pull request commentfyne-io/fyne-cross

Support specifying target architectures for Android

I wonder if it needs to be more specific about what universal is - the docs simply say multiple architectures. Or maybe not, I guess this is platform specific. Good point. I'll update to use multiple architectures.

Does it need to be applied to iOS and macOS as well? That would be great. Wondering if we should implement this feature into the fyne cli tool.

lucor

comment created time in 6 days

PR opened fyne-io/fyne-cross

Support specifying target architectures for Android

This PR adds the possibility to specify a target architecture for Android via the -arch flag. It also introduces the universal architecture constant.

Fixes #52

+200 -141

0 comment

6 changed files

pr created time in a month

create barnchlucor/fyne-io-fyne-cross

branch : feature/android-architectures

created branch time in a month

issue commentfyne-io/fyne-cross

Compile to andriod error

It is difficult to debug with these info. Could you please provide a minimal example that reproduces the problem? Additionally are you by any chance using a third party lib ? If so, please double check if that lib is compatible with android.

xfengw

comment created time in a month

delete branch lucor/fyne-io-fyne-cross

delete branch : feature/docker-go-1.17-android-30

delete time in a month

push eventfyne-io/fyne-cross

lucor

commit sha 258f9f92488ef07850f8decbf731d5e71c601522

Update Go to v1.17.0

view details

lucor

commit sha cfad0ba6fee98083d9433bc5137bfbfc10b22fae

Update Android builds to SDK 30

view details

lucor

commit sha 06159976645564976567a18e8d12a00cd6ef1c44

Revert "Update Go to v1.17.0" This reverts commit 258f9f92488ef07850f8decbf731d5e71c601522.

view details

Luca Corbo

commit sha 6b4bc88fa5e1e26b4ef69f63e5dcc44630316f52

Merge pull request #63 from lucor/feature/docker-go-1.17-android-30 Update docker images: Go v1.17 and Android builds SDK 30

view details

push time in a month

PR merged fyne-io/fyne-cross

Update docker images: Go v1.17 and Android builds SDK 30

This PR updates the android docker image to use the latest command line tools and updates the Android builds to SDK 30

+11 -9

2 comments

1 changed file

lucor

pr closed time in a month

Pull request review commentfyne-io/fyne-cross

Propose the ability to use podman

 type Options struct { 	Debug        bool     // Debug if true enable log verbosity } +// initIsPodman returns true if docker is an alias to podman (e.g. via podman-docker).+func initIsPodman() bool {+	// read the "docker" executable to check if it's a podman wrapper++	if os.Getenv("FYNECROSS_FORCE_PODMAN") == "true" {

I see the pro to force an engine. If you agree I'd move to a fyne-cross flag like --engine=<type> where the possibile values are: auto, docker, podman (default to auto-detect)

metal3d

comment created time in a month

Pull request review commentfyne-io/fyne-cross

Propose the ability to use podman

 func goBuild(ctx Context) error {  	// add ldflags to command, if any 	if len(ldflags) > 0 {-		args = append(args, "-ldflags", fmt.Sprintf("'%s'", strings.Join(ldflags, " ")))+		flags := make([]string, len(ldflags))

It seems passing multiple flags does not work: fyne-cross linux -ldflags="-X=main.Name=Podman -X=main.Title=Test" --debug

/usr/bin/docker run --rm -t -w /app -v /code/lucor/test:/app:z -v /home/lucor/.cache/fyne-cross:/go:z -e CGO_ENABLED=1 -e GOCACHE=/go/go-build -e GOOS=linux -e GOARCH=amd64 -e CC=gcc -e GOFLAGS= -ldflags=-w -ldflags=-s -e fyne_uid=1000 docker.io/fyneio/fyne-cross:1.1-base go build -ldflags=-X=-X=main.Title=Test -w -s -o /app/fyne-cross/bin/linux-amd64/test -v .
flag provided but not defined: -w
usage: go build [-o output] [build flags] [packages]
Run 'go help build' for details.
[✗] exit status 2

If you agree, I'd add some tests on this part of logic to ensure we won't break any previous command relying on the fyne-cross 'ldflags` or GOFLAGS env variable.

metal3d

comment created time in a month

Pull request review commentfyne-io/fyne-cross

Propose the ability to use podman

 type Options struct { 	Debug        bool     // Debug if true enable log verbosity } +// initIsPodman returns true if docker is an alias to podman (e.g. via podman-docker).+func initIsPodman() bool {

What about to add a function that returns the engine in use, and optionally some helper isXXX functions ? Example: func engine() (string, error) where string can be docker or podman func isEngineDocker() bool func isEnginePodman() bool

This will allow to remove the init function and add support for new engines.

metal3d

comment created time in a month

PullRequestReviewEvent
PullRequestReviewEvent

pull request commentfyne-io/fyne-cross

Update docker images: Go v1.17 and Android builds SDK 30

@Jacalz it is a good point. Yes, the idea was to add support for windows/arm64 as well but let's remove the Go 1.17 update from this PR (done in 0615997) and handle later with a dedicated one :)

lucor

comment created time in a month

push eventlucor/fyne-io-fyne-cross

lucor

commit sha 06159976645564976567a18e8d12a00cd6ef1c44

Revert "Update Go to v1.17.0" This reverts commit 258f9f92488ef07850f8decbf731d5e71c601522.

view details

push time in a month

PR opened fyne-io/fyne-cross

Update docker images: Go v1.17 and Android builds SDK 30

This PR updates the following docker images:

  • base to use Go version 1.17.0
  • android to use the latest command line tools and updates the Android builds to SDK 30
+36 -28

0 comment

10 changed files

pr created time in a month

create barnchlucor/fyne-io-fyne-cross

branch : feature/docker-go-1.17-android-30

created branch time in a month

push eventfyne-io/fyne-cross

Jacob Alzen

commit sha 60ceabad68d5a62cbfdae549ee1bea7c37508465

Switch to x/sys/execabs for windows security fixes The os/exec package on Windows will match the behaviour of cmd.exe by considering the local folder as a primary part of the path. This means that a malicious binary with the same name, in the current folder, would be run instead of the expected binary in the system path. Due to the backwards compat being an issue, this could not be fixed within ox/exec before Go v2. See https://blog.golang.org/path-security for more info.

view details

Luca Corbo

commit sha 21ee353925e19ffdb6ab5b3df94a7ecbc150c25b

Merge pull request #57 from Jacalz/execabs Switch to x/sys/execabs for windows security fixes

view details

push time in a month

PR merged fyne-io/fyne-cross

Switch to x/sys/execabs for windows security fixes

The os/exec package on Windows will match the behaviour of cmd.exe by considering the local folder as a primary part of the path. This means that a malicious binary with the same name, in the current folder, would be run instead of the expected binary in the system path. Due to the backwards compat being an issue, this could not be fixed within ox/exec before Go v2. See https://blog.golang.org/path-security for more info.

Related to https://github.com/fyne-io/fyne/pull/2344 but not part of the same effort.

+20 -17

0 comment

7 changed files

Jacalz

pr closed time in a month

PullRequestReviewEvent

delete branch lucor/fyne-io-fyne-cross

delete branch : feature/go1.16.5

delete time in a month

PullRequestReviewEvent