Skip to content

lmp-boot-firmware: cleanup and minor changes#1145

Merged
ricardosalveti merged 4 commits intofoundriesio:mainfrom
quaresmajose:lmp-boot-firmware
Jun 14, 2023
Merged

lmp-boot-firmware: cleanup and minor changes#1145
ricardosalveti merged 4 commits intofoundriesio:mainfrom
quaresmajose:lmp-boot-firmware

Conversation

@quaresmajose
Copy link
Copy Markdown
Member

No description provided.

@quaresmajose quaresmajose requested a review from a team May 9, 2023 09:47
Comment thread meta-lmp-base/recipes-bsp/lmp-boot-firmware/lmp-boot-firmware.bb
@ricardosalveti
Copy link
Copy Markdown
Member

Can you please extend the commit message on every change to avoid having changes with subject only descriptions?

Copy link
Copy Markdown
Contributor

@igoropaniuk igoropaniuk left a comment

Choose a reason for hiding this comment

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

LGTM

Modify content on the work or source dir is not safe and can result in build errors

Signed-off-by: Jose Quaresma <[email protected]>
We can store the full path on FIRMWARE_DEPLOY_DIR to avoid contatenation

Signed-off-by: Jose Quaresma <[email protected]>
Instaed of deploying each file one by one we can deploy the full dir

Signed-off-by: Jose Quaresma <[email protected]>
@quaresmajose
Copy link
Copy Markdown
Member Author

Can you please extend the commit message on every change to avoid having changes with subject only descriptions?

Added descriptions in all except the indentation fix

Copy link
Copy Markdown
Contributor

@MrCry0 MrCry0 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@ricardosalveti ricardosalveti left a comment

Choose a reason for hiding this comment

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

LGTM

@ricardosalveti ricardosalveti merged commit afbee17 into foundriesio:main Jun 14, 2023
@quaresmajose quaresmajose deleted the lmp-boot-firmware branch June 19, 2023 13:17
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.

5 participants