Skip to content
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

Tools: Fix Python3 + firmware header traeback #8330

Merged
merged 1 commit into from
Oct 9, 2018

Commits on Oct 4, 2018

  1. Tools: Fix Python3 + firmware header traeback

    ### Description
    
    Traceback:
    ```
    Merging Regions
      Filling region bootloader with mbed-cloud-client-example/mbed-os/features/FEATURE_BOOTLOADER/targets/TARGET_Freescale/TARGET_MCUXpresso_MCUS/TARGET_MCU_K64F/TARGET_FRDM/mbed-bootloader-k64f-block_device-sotp-v3_4_0.bin
      Padding region bootloader with 0x9a4 bytes
    Traceback (most recent call last):
      File "mbed-cloud-client-example/mbed-os/tools/make.py", line 293, in <module>
        ignore=options.ignore
      File "mbed-cloud-client-example/mbed-os/tools/build_api.py", line 548, in build_project
        merge_region_list(region_list, res, notify)
      File "mbed-cloud-client-example/mbed-os/tools/build_api.py", line 423, in merge_region_list
        _fill_header(region_list, region).tofile(header_filename, format='hex')
      File "mbed-cloud-client-example/mbed-os/tools/build_api.py", line 380, in _fill_header
        header.puts(start, struct.pack(fmt, time()))
    struct.error: required argument is not an integer
    ```
    
    reason: `time()` returns a float. So the fix is to force it to be an `int`.
    
    ### Pull request type
    
        [x] Fix
        [ ] Refactor
        [ ] Target update
        [ ] Functionality change
        [ ] Breaking change
    theotherjimmy committed Oct 4, 2018
    Configuration menu
    Copy the full SHA
    8d63ef6 View commit details
    Browse the repository at this point in the history