Skip to content

images: Use a regular file, not a symbolic link, for README.md#723

Merged
debarshiray merged 1 commit intocontainers:masterfrom
debarshiray:wip/rishi/images-remove-symbolic-links
Mar 15, 2021
Merged

images: Use a regular file, not a symbolic link, for README.md#723
debarshiray merged 1 commit intocontainers:masterfrom
debarshiray:wip/rishi/images-remove-symbolic-links

Conversation

@debarshiray
Copy link
Copy Markdown
Member

A while ago, 'podman build' stopped supporting COPY with relative
symbolic links [1]. Therefore, these image definitions can't be used
without first temporarily removing the symbolic links, which is
annoying.

The downside is that the copies of README.md now has to be separately
updated, which isn't that big of a hassle compared to the problem that
it fixes.

[1] containers/buildah#1952

A while ago, 'podman build' stopped supporting COPY with relative
symbolic links [1]. Therefore, these image definitions can't be used
without first temporarily removing the symbolic links, which is
annoying.

The downside is that the copies of README.md now has to be separately
updated, which isn't that big of a hassle compared to the problem that
it fixes.

[1] containers/buildah#1952

containers#723
@debarshiray debarshiray force-pushed the wip/rishi/images-remove-symbolic-links branch from b34dc5d to a8c6fd1 Compare March 15, 2021 15:46
@martymichal martymichal added 2. Documentation The issue is related to the user documentation 3. Bugfix Fixes a bug labels Mar 15, 2021
@softwarefactory-project-zuul
Copy link
Copy Markdown

Build failed.

@debarshiray debarshiray merged commit a8c6fd1 into containers:master Mar 15, 2021
@debarshiray debarshiray deleted the wip/rishi/images-remove-symbolic-links branch March 15, 2021 16:01
@debarshiray
Copy link
Copy Markdown
Member Author

Thanks, @HarryMichal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. Documentation The issue is related to the user documentation 3. Bugfix Fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants