Skip to content

[build] Ensure /usr/lib/systemd/system/ directory exists before referencing#4788

Merged
abdosi merged 2 commits intosonic-net:masterfrom
abdosi:systemd_compile_fix
Jun 17, 2020
Merged

[build] Ensure /usr/lib/systemd/system/ directory exists before referencing#4788
abdosi merged 2 commits intosonic-net:masterfrom
abdosi:systemd_compile_fix

Conversation

@abdosi
Copy link
Contributor

@abdosi abdosi commented Jun 17, 2020

- Why I did it
Fix the Build Failure on 201911 (Stretch) because of directory /usr/lib/systemd/system/ does not exist.creating manually the directory as part of build image.
Change should not harm Master (buster) where the directory is created by Linux

- How I did it
using mkdir -p

- How to verify it
Verified 201911 Build is fine after the change

/usr/lib/systemd/system/ does not exist so creating
manually. Change should not harm Master (buster) where
the directory is created by Linux
@jleveque jleveque changed the title Fix the Build Failure on 201911 (Stretch) because of directory /usr/lib/systemd/system/ not present [build] Ensure /usr/lib/systemd/system/ directory exists before referencing Jun 17, 2020
@jleveque
Copy link
Contributor

@abdosi: FYI, I updated the PR title.

@abdosi abdosi force-pushed the systemd_compile_fix branch from 10da96c to 46bbaec Compare June 17, 2020 01:18
@abdosi abdosi force-pushed the systemd_compile_fix branch from 46bbaec to 11a8393 Compare June 17, 2020 01:20
@abdosi abdosi merged commit 30d7ce0 into sonic-net:master Jun 17, 2020
@abdosi abdosi deleted the systemd_compile_fix branch June 17, 2020 16:17
abdosi added a commit that referenced this pull request Jun 17, 2020
…encing (#4788)

* Fix the Build on 201911 (Stretch) where the directory
/usr/lib/systemd/system/ does not exist so creating
manually. Change should not harm Master (buster) where
the directory is created by Linux

* Fix as per review comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants