Project

General

Profile

Actions

Bug #431

closed

[PATCH] Modify get_pkg_depends to return whole paths

Added by znerol znerol about 14 years ago. Updated about 14 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Package System
Target version:
Start date:
03/17/2010
Due date:
% Done:

0%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
All
Affected Architecture:

Description

The function sync_package from /etc/inc/pkg-utils.inc ensures that all required files from a particular package are present on system startup. The list of files a package depends on is compiled by get_pkg_depends which parses the xml configuration. Unfortunately the current implementation always looks for the dependency files in /usr/local/pkg and does not consider the optional prefix. Therefore a package containing dependency files with a non-standard prefix will always get reinstalled on each bootup. The attached logfiles illustrate this behaviour.

The attached patch fixes the functions sync_package and get_pkg_depends.


Files

0001-Modify-get_pkg_depends-to-return-whole-paths.patch (2.5 KB) 0001-Modify-get_pkg_depends-to-return-whole-paths.patch znerol znerol, 03/17/2010 09:20 AM
console-log-before.txt (1.98 KB) console-log-before.txt znerol znerol, 03/17/2010 09:20 AM
system-log-before.txt (2.36 KB) system-log-before.txt znerol znerol, 03/17/2010 09:20 AM
console-log-after.txt (224 Bytes) console-log-after.txt znerol znerol, 03/17/2010 09:20 AM
system-log-after.txt (693 Bytes) system-log-after.txt znerol znerol, 03/17/2010 09:20 AM
Actions #1

Updated by Scott Ullrich about 14 years ago

  • Status changed from New to Feedback

Commited, thanks! Please let us know if it is resolved on the next snapshot.

Actions #2

Updated by Chris Buechler about 14 years ago

  • Target version set to 2.0
Actions #3

Updated by znerol znerol about 14 years ago

Tested with pfSense-2.0-BETA1-20100317-2238.iso and tinydns pkg. It works, thanks.

Actions #4

Updated by Chris Buechler about 14 years ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF