Add debian packaging

Conversation 2 Commits 7 Files Changed 9

osmith commented 3 months ago

Collaborator

Related: SYS#5538

osmith added 2 commits 3 months ago

efa74b0d02 debian: add dir for packaging

Related: SYS#5538

189a39639d gitignore: new file

osmith added 3 commits 3 months ago

1bf48da7a8 Fix missing pci_fill_info()

Before reading vendor_id or device_id from struct pci_dev, one must call
pci_fill_info() first. It looks like this had worked with previous
libpci versions, but with libpci 3.7.0 on debian 11 this patch is needed
or else the tool stops at 'starting bus iteration' without finding the
device.

Fixes: SYS#5956

313de8d95e Exit with error if no device was found

347924cc08 d/changelog: add 1.0.1

osmith changed title from ~~Add debian packaging~~ to Draft: Add debian packaging 3 months ago

osmith changed title from ~~Draft: Add debian packaging~~ to WIP: Add debian packaging 3 months ago

hwelte commented 3 months ago

Owner

looks good to me. Not merging as i's still WIP.

osmith added 2 commits 3 months ago

4efcedfeca Query PCI_FILL_{BASES,SIZES} in pci_fill_info too

The previous fix for missing pci_fill_info() was incomplete, it caused
the device to be found but changing the mac did not succeed. Fix it up
as separate commit, as the other commit is already packaged.

630734375f d/changelog: add 1.0.2

osmith changed title from ~~WIP: Add debian packaging~~ to Add debian packaging 3 months ago

osmith commented 3 months ago

Poster

Collaborator

The fix was incomplete, but now it works. WIP removed.

hwelte merged commit dbac2a5254 into master 3 months ago

hwelte referenced this issue from a commit 3 months ago

Merge pull request 'Add debian packaging' (#1) from osmith/rtl8168-eeprom:osmith/debian into master

osmith deleted branch osmith/debian 3 months ago

The pull request has been merged as dbac2a5254.

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Add debian packaging #1