Skip to content

Arduino: add Portfile for Arduino 1.8.8 #3667

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 21, 2019
Merged

Arduino: add Portfile for Arduino 1.8.8 #3667

merged 1 commit into from
Feb 21, 2019

Conversation

giansalvo
Copy link
Contributor

Description

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 10.13.4
Xcode 9.3.1

Verification

Have you

  • checked your Portfile with port lint?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?

@macportsbot
Copy link

Travis Build #5279 Failed.

Lint results
--->  Verifying Portfile for Arduino
--->  0 errors and 0 warnings found.

Port Arduino fail on xcode10.2. Log
Port Arduino fail on xcode9.4. Log
Port Arduino success on xcode7.3. Log

Copy link
Member

@mojca mojca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't take a closer look, but quite some fetching of files during build is going on. Is there any way to avoid that and pre-fetch & checksum all the files?

I don't want to push too hard because building this one is definitely not among the easiest ports. I just added some random trivial comments, it would be cool to get some feedback from more experienced users. Ihave absolutely no idea how difficult it is to actually build the Arduino IDE, I never tried it myself (I only used the downloaded binary so far).

@g5pw: do you have any comments / ideas about this one?

}

destroot {
system -W ${worksrcpath}/build "ant build dist -Dplatform=macosx <<<'${version}'"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would move this one to the build phase (and probably modify it a bit, perhaps something in the sense of build.cmd ant etc, but that's of secondary importance).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I understand correctly, the build command is issued in the working directory. In this case the build command (ant) must run in ${worksrcpath}/build. How can I do this?

@mojca
Copy link
Member

mojca commented Feb 16, 2019

Btw: the build failures come from

java.nio.file.NoSuchFileException: /Library/Java/JavaVirtualMachines/jdk-10.0.1.jdk/MacOS

@giansalvo
Copy link
Contributor Author

Btw: the build failures come from

java.nio.file.NoSuchFileException: /Library/Java/JavaVirtualMachines/jdk-10.0.1.jdk/MacOS

Thanks. The Arduino documentation says:

if you also have JDK 9 or 10 installed, you will need to select JDK 8

How can I set a dependency from JDK8?

@macportsbot
Copy link

Travis Build #5280 Failed.

Lint results
--->  Verifying Portfile for Arduino
--->  0 errors and 0 warnings found.

Port Arduino fail on xcode10.2. Log
Port Arduino fail on xcode9.4. Log
Port Arduino success on xcode7.3. Log

@giansalvo
Copy link
Contributor Author

This version fails because Arduino 1.8.8 require JavaFX to compile which is not available in openjdk (it's available only in Oracle JDK). There is an open version called openjfx which is not available on MacPorts. Recently upstream modified the code to use jtouchbar 1.0.0 which is available open source, but I'm not able to integrate in the port process. I live here some references for someone that would like to help.
arduino/Arduino#8255
arduino/Arduino#8156
arduino/Arduino#8417
arduino/Arduino#8429
arduino/Arduino#8156

@macportsbot
Copy link

Travis Build #5286 Failed.

Lint results
--->  Verifying Portfile for Arduino
--->  0 errors and 0 warnings found.

Port Arduino fail on xcode10.2. Log
Port Arduino fail on xcode9.4. Log
Port Arduino fail on xcode8.3. Log

@macportsbot
Copy link

Travis Build #5303 Failed.

Lint results
--->  Verifying Portfile for Arduino
--->  0 errors and 0 warnings found.

Port Arduino fail on xcode10.2. Log
Port Arduino fail on xcode9.4. Log
Port Arduino fail on xcode8.3. Log
Port Arduino fail on xcode7.3. Log

@macportsbot
Copy link

Travis Build #5331 Passed.

Lint results
--->  Verifying Portfile for Arduino
--->  0 errors and 0 warnings found.

Port Arduino success on xcode10.2. Log
Port Arduino success on xcode9.4. Log
Port Arduino success on xcode8.3. Log
Port Arduino success on xcode7.3. Log

@macportsbot
Copy link

Travis Build #5336 Passed.

Lint results
--->  Verifying Portfile for Arduino
--->  0 errors and 0 warnings found.

Port Arduino success on xcode10.2. Log
Port Arduino success on xcode9.4. Log
Port Arduino success on xcode8.3. Log
Port Arduino success on xcode7.3. Log

Copy link
Member

@mojca mojca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Someone would usually ask you to squash the commits, but I did that now.

Also, since there is no single other port in the electronics, I moved it to devel. I don't know if this is the best category, but if we introduce electronics, it would only make sense to do so if we move a bunch of other ports there as well.

While squashing the commits I probably screwed up with commit author's email identity. I would have merged this already, but maybe you can give your final thumbs up or comments if I should fix something myself. Other than that it looks ready to go.

@macportsbot
Copy link

Travis Build #5339 Passed.

Lint results
--->  Verifying Portfile for Arduino
--->  0 errors and 0 warnings found.

Port Arduino success on xcode10.2. Log
Port Arduino success on xcode9.4. Log
Port Arduino success on xcode8.3. Log
Port Arduino success on xcode7.3. Log

@mojca
Copy link
Member

mojca commented Feb 21, 2019

I would have one further objection, but I don't want to go too deep into nitpicking, and my suggestion would be to try to fix this later, else it might take "forever" to get this in. This port is downloading files during the build, which is kind-of against our general policy to first fetch everything in fetch phase, then checksum it, and finally build without any further downloads. I would vote for accepting this, but trying to follow the policy more closely some time soon in the future.

I also suspect that the supported architectures and OS versions are limited to something like intel & >= 10.7, but I didn't test anything.

@giansalvo
Copy link
Contributor Author

I agree to merge the changes and after I'll investigate the fetch problem and the supported systems.

@mojca mojca merged commit 7b8dab4 into macports:master Feb 21, 2019
@giansalvo giansalvo deleted the add_Arduino_1_8_8 branch February 22, 2019 07:33
@mojca mojca mentioned this pull request Feb 24, 2019
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants