[SRU] update-pciids in pciutils fails to create hard link
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
pciutils (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Focal |
Fix Released
|
High
|
Erich Eickmeyer |
Bug Description
[Impact]
* People who use update-pciids more than once are unable to run it a second time without errors due to a missing "-f" in the "ln" line being called as the symlink already exists. This was identified upstream and fixed in later versions, but was not implemented in Focal. The patch in question is fixed in a new upload and attached patch.
[Test Plan]
* Install pciutils
* Run 'sudo update-pciids'
* Run 'sudo update-pciids' a second time.
* "ln" will return an error explaining the hardlink already exists.
[Where problems could occur]
* If this patch is in error, then the entire command will cause the same problem or the link of the '.old' file will be erased entirely. This is highly unlikely.
[Other Info]
* Corrected patch is attached to this bug.
------
Original bug report:
Running on Ubuntu 20.04 LTS, the `.old` file does not seem to be cleaned up properly.
It seems someone forgot to add the "-f" from the respective upstream commit: https:/
First run:
```
$ sudo sh -x /usr/sbin/
+ set -e
+ SRC=https:/
+ DEST=/usr/
+ PCI_COMPRESSED_IDS=
+ GREP=grep
+ [ = -q ]
+ quiet=false
+ touch /usr/share/
+ [ = 1 ]
+ which bzip2
+ DECOMP=bzip2 -d
+ SRC=https:/
+ which curl
+ DL=curl -o /usr/share/
+ false
+ curl -o /usr/share/
% Total % Received % Xferd Average Speed Time Time Time Current
100 265k 100 265k 0 0 634k 0 --:--:-- --:--:-- --:--:-- 635k
+ bzip2 -d
+ grep ^C /usr/share/
+ [ -f /usr/share/
+ ln /usr/share/
+ chmod -f --reference=
+ mv /usr/share/
+ rm /usr/share/
+ [ /usr/share/
+ false
+ echo Done.
Done.
```
Second run:
```
$ sudo sh -x /usr/sbin/
+ set -e
+ SRC=https:/
+ DEST=/usr/
+ PCI_COMPRESSED_IDS=
+ GREP=grep
+ [ = -q ]
+ quiet=false
+ touch /usr/share/
+ [ = 1 ]
+ which bzip2
+ DECOMP=bzip2 -d
+ SRC=https:/
+ which curl
+ DL=curl -o /usr/share/
+ false
+ curl -o /usr/share/
% Total % Received % Xferd Average Speed Time Time Time Current
100 265k 100 265k 0 0 420k 0 --:--:-- --:--:-- --:--:-- 419k
+ bzip2 -d
+ grep ^C /usr/share/
+ [ -f /usr/share/
+ ln /usr/share/
ln: failed to create hard link '/usr/share/
```
affects: | ubuntu → pciutils (Ubuntu) |
tags: | added: rls-ff-incoming |
tags: | added: fr-1144 |
tags: | removed: rls-ff-incoming |
Changed in pciutils (Ubuntu Focal): | |
importance: | Undecided → High |
status: | New → Triaged |
description: | updated |
summary: |
- update-pciids in pciutils fails to create hard link + [SRU] update-pciids in pciutils fails to create hard link |
Changed in pciutils (Ubuntu Focal): | |
status: | Triaged → In Progress |
assignee: | nobody → Erich Eickmeyer (eeickmeyer) |
Thank you for taking the time to report this bug and helping to make Ubuntu better. It seems that your bug report is not filed about a specific source package though, rather it is just filed against Ubuntu in general. It is important that bug reports be filed about source packages so that people interested in the package can find the bugs about it. You can find some hints about determining what package your bug might be about at https:/ /wiki.ubuntu. com/Bugs/ FindRightPackag e. You might also ask for help in the #ubuntu-bugs irc channel on Freenode.
To change the source package that this bug is filed about visit https:/ /bugs.launchpad .net/ubuntu/ +bug/1915923/ +editstatus and add the package name in the text box next to the word Package.
[This is an automated message. I apologize if it reached you inappropriately; please just reply to this message indicating so.]