Skip to contentSkip to conversation

Uprev prelink (v2)

PATCH In review Mark Hatle 2 participants 4 messages Jun 24, 2011 – Jun 25, 2011

lore
Share & tools
lore

Mark Hatle

author
#0 lore raw

cover Uprev prelink (v2)

Updated the prelink to latest version.

Also split out the cron configuration, change the configuration so it's enabled by default.

The following changes since commit 3aec2fa2df9aaa883feda0d7aed85e63d01398b9:

qemuimagetest: update cvs and iptables to newer version for toolchain test (2011-06-24 11:28:28 +0100)

are available in the git repository at:
git://git.pokylinux.org/poky-contrib mhatle/prelink
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=mhatle/prelink

Mark Hatle (1):
prelink: Uprev prelink to latest version

 .../prelink/prelink/prelink.default                |    2 +-
 meta/recipes-devtools/prelink/prelink_git.bb       |   25 +++++++++++++++++--
 2 files changed, 23 insertions(+), 4 deletions(-)
Signature · 1 line
1.7.3.4
author
#1 lore raw

PATCH 1/1 prelink: Uprev prelink to latest version (V2)

The latest cross-prelink version tracks the prelink r190 SVN.

Fix two minor issues as well. If we remove the package, we would unprelink the filesystem after the rm was finished. This is incorrect.. we need to do this prerm.

Move the cron and cron configuration file to a new package split. This item is not valid for most embedded users.

---
 .../prelink/prelink/prelink.default                |    2 +-
 meta/recipes-devtools/prelink/prelink_git.bb       |   25 +++++++++++++++++--
 2 files changed, 23 insertions(+), 4 deletions(-)
meta/recipes-devtools/prelink/prelink/prelink.default +1 -1
index 901258e..d8802f8 100644
@@ -2,7 +2,7 @@
 # or to yes to enable it.
 # (if you change this from yes to no prelink -ua
 # will be run next night to undo prelinking)
-PRELINKING=unknown
+PRELINKING=yes
 
 # Options to pass to prelink
 # -m	Try to conserve virtual memory by allowing overlapping
meta/recipes-devtools/prelink/prelink_git.bb +22 -3
index 1eea30a..b57c145 100644
@@ -8,9 +8,24 @@ and executables, so that far fewer relocations need to be resolved at \
 runtime and thus programs come up faster."
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=c93c0550bd3173f4504b2cbd8991e50b"
-SRCREV = "909470ee441237563d6236c505cb2d02ddc48704"
+SRCREV = "ac461e73b17253a4da25c5aafeac7193b553156c"
 PV = "1.0+git${SRCPV}"
-PR = "r2"
+PR = "r3"
+
+#
+# The cron script attempts to re-prelink the system daily -- on
+# systems where users are adding applications, this might be reasonable
+# but for embedded, we should be re-running prelink -a after an update.
+#
+# Default is prelinking is enabled.
+#
+SUMMARY_${PN}-cron = "Cron scripts to control automatic prelinking"
+DESCRIPTION_${PN}-cron = "Cron scripts to control automatic prelinking.  \
+See: ${sysconfdir}/cron.daily/prelink for configuration information."
+
+FILES_${PN}-cron = "${sysconfdir}/cron.daily ${sysconfdir}/default"
+
+PACKAGES =+ "${PN}-cron"
 
 SRC_URI = "git://git.yoctoproject.org/prelink-cross.git;protocol=git \
            file://prelink.conf \
@@ -53,9 +68,13 @@ fi
 prelink -a
 }
 
-pkg_postrm_prelink() {
+pkg_prerm_prelink() {
 #!/bin/sh
 
+if [ "x$D" != "x" ]; then
+  exit 1
+fi
+
 prelink -au
 }
 
Signature · 1 line
1.7.3.4
#2 lore raw

Re: [PATCH 1/1] prelink: Uprev prelink to latest version (V2)

Quoted 20 lines · Mark Hatle

The latest cross-prelink version tracks the prelink r190 SVN.

Fix two minor issues as well. If we remove the package, we would unprelink the filesystem after the rm was finished. This is incorrect.. we need to do this prerm.

Move the cron and cron configuration file to a new package split. This item is not valid for most embedded users.

---
 .../prelink/prelink/prelink.default                |    2 +-
 meta/recipes-devtools/prelink/prelink_git.bb       |   25 +++++++++++++++++--
 2 files changed, 23 insertions(+), 4 deletions(-)
meta/recipes-devtools/prelink/prelink/prelink.default +0 -0
index 901258e..d8802f8 100644

meta/recipes-devtools/prelink/prelink/prelink.default line 6

Quoted 6 lines
meta/recipes-devtools/prelink/prelink/prelink.default +1 -1
@@ -2,7 +2,7 @@
 # or to yes to enable it.
 # (if you change this from yes to no prelink -ua
 # will be run next night to undo prelinking)
-PRELINKING=unknown
+PRELINKING=yes
 

this is a functionality change. Will it have system wide effects ?

author
#3 lore raw

Re: [PATCH 1/1] prelink: Uprev prelink to latest version (V2)

Quoted 16 lines · Khem Raj
Quoted 15 lines · Mark Hatle

The latest cross-prelink version tracks the prelink r190 SVN.

Fix two minor issues as well. If we remove the package, we would unprelink the filesystem after the rm was finished. This is incorrect.. we need to do this prerm.

Move the cron and cron configuration file to a new package split. This item is not valid for most embedded users.

---
 .../prelink/prelink/prelink.default                |    2 +-
 meta/recipes-devtools/prelink/prelink_git.bb       |   25 +++++++++++++++++--
 2 files changed, 23 insertions(+), 4 deletions(-)
Quoted 13 lines

meta/recipes-devtools/prelink/prelink/prelink.default line 6

Quoted 11 lines
meta/recipes-devtools/prelink/prelink/prelink.default +1 -1
index 901258e..d8802f8 100644
@@ -2,7 +2,7 @@
 # or to yes to enable it.
 # (if you change this from yes to no prelink -ua
 # will be run next night to undo prelinking)
-PRELINKING=unknown
+PRELINKING=yes
 

this is a functionality change. Will it have system wide effects ?

If the user previously had the prelinker installed onto the target, and had the cron system installed to run the daily scripts, and never bothered to configure prelink before.. whenever the script ran it would undo any prelinking that had occurred.

After the change, it's now optional to install the cron scripts. If they are installed, they default to -on- which means that if cron runs the script it will attempt to update the prelinking on the system.

This does not affect the build-time prelinking in any way. For many systems there is no reason to have the prelinker on the target, it's only needed for field upgrades.

So, I consider this an enhancement (splitting the script out to make it optional), and a bug-fix -- enabling re-prelink functionality if the script is installed -- vs the previous "un-prelink" being the default.

--Mark

Quoted 4 lines

_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core