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

The "debian" folder of a newly added platform driver is ignored to be uploaded. #7683

Open
Komil-yc opened this issue May 24, 2021 · 3 comments
Assignees
Labels
Triaged this issue has been triaged

Comments

@Komil-yc
Copy link

Description

There is a pattern "platform/**/debian/*" described in .gitignore.

That causes GIT to ignore all files in debian/ used to build Debian package.
A newly added platform will miss debian/ folder to finish their building.
The change is introduced by below PR:
#4803

Is that an expected behavior to ignore debian/ folder automatically and
should we upload them by another way like "git add -f" ?

Steps to reproduce the issue:

Describe the results you received:

Describe the results you expected:

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

@anshuv-mfst
Copy link

Hi @jleveque - could you please take a look, thanks.

@jleveque
Copy link
Contributor

I agree that this ignore is a bit severe, and could cause vendors to miss adding their debian directory. I will open a PR to remove this line. However, I think each vendor should be responsible for providing one or more .gitignore files to properly ignore generated files in their platform directories.

@zhangyanzhao zhangyanzhao added the Triaged this issue has been triaged label Jun 23, 2021
bluecmd added a commit to bluecmd/sonic-buildimage that referenced this issue Apr 7, 2023
Includes the common debian files that we always want to include.

This mitigates but does not fully solve sonic-net#7683 as it
could be more files that are ignored by this default rule.

Signed-off-by: Christian Svensson <blue@cmd.nu>
@bluecmd
Copy link
Contributor

bluecmd commented Apr 7, 2023

I created a PR to mitigate this issue as I lost a few hours today because of files being hidden from git status; #14565

lguohan pushed a commit that referenced this issue Apr 12, 2023
Includes the common debian files that we always want to include.

This mitigates but does not fully solve #7683 as it
could be more files that are ignored by this default rule.

Signed-off-by: Christian Svensson <blue@cmd.nu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Triaged this issue has been triaged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants