Skip to content

providers/vmware: add missing public functions for non-amd64#1022

Merged
jlebon merged 1 commit intocoreos:mainfrom
dongsupark:vmware-arm64-unsupported
Jan 10, 2024
Merged

providers/vmware: add missing public functions for non-amd64#1022
jlebon merged 1 commit intocoreos:mainfrom
dongsupark:vmware-arm64-unsupported

Conversation

@dongsupark
Copy link
Contributor

In v5.5.0, VmwareProvider must have a corresponding empty function parse_netplan_config() in unsupported part as well, just like the amd64 part.

Otherwise build would fail when cross-building afterburn for arm64.

error[E0599]: no method named `parse_netplan_config` found for reference
`&VmwareProvider` in the current scope
  --> src/providers/vmware/mod.rs:37:14
   |
37 |         self.parse_netplan_config()
   |              ^^^^^^^^^^^^^^^^^^^^ method not found in
`&VmwareProvider`

See also #888, flatcar/scripts#1467.

VmwareProvider must have a corresponding empty function
`parse_netplan_config()` in unsupported part as well, just like the
amd64 part.
Otherwise build would fail when cross-building afterburn for arm64.

```
error[E0599]: no method named `parse_netplan_config` found for reference
`&VmwareProvider` in the current scope
  --> src/providers/vmware/mod.rs:37:14
   |
37 |         self.parse_netplan_config()
   |              ^^^^^^^^^^^^^^^^^^^^ method not found in
`&VmwareProvider`
```
@dongsupark dongsupark force-pushed the vmware-arm64-unsupported branch from e1324bf to 13f3dd5 Compare December 5, 2023 12:21
Copy link
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

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

Thanks!

@jlebon jlebon merged commit bbdaa4f into coreos:main Jan 10, 2024
@mike-nguyen mike-nguyen mentioned this pull request Jan 12, 2024
55 tasks
dongsupark added a commit to flatcar/scripts that referenced this pull request Jan 15, 2024
Update afterburn to 5.5.1, and remove the patch 0003 for fixing
cross-build issue of arm64, because the upstream fix
coreos/afterburn#1022 is already included in
5.5.1.
dongsupark added a commit to flatcar/scripts that referenced this pull request Jan 17, 2024
Update afterburn to 5.5.1, and remove the patch 0003 for fixing
cross-build issue of arm64, because the upstream fix
coreos/afterburn#1022 is already included in
5.5.1.
dongsupark added a commit to flatcar/scripts that referenced this pull request Jan 17, 2024
Update afterburn to 5.5.1, and remove the patch 0003 for fixing
cross-build issue of arm64, because the upstream fix
coreos/afterburn#1022 is already included in
5.5.1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants