summaryrefslogtreecommitdiffstats
path: root/zarb-ml/mageia-dev/2011-June/005397.html
diff options
context:
space:
mode:
Diffstat (limited to 'zarb-ml/mageia-dev/2011-June/005397.html')
-rw-r--r--zarb-ml/mageia-dev/2011-June/005397.html200
1 files changed, 200 insertions, 0 deletions
diff --git a/zarb-ml/mageia-dev/2011-June/005397.html b/zarb-ml/mageia-dev/2011-June/005397.html
new file mode 100644
index 000000000..6dfaaaad9
--- /dev/null
+++ b/zarb-ml/mageia-dev/2011-June/005397.html
@@ -0,0 +1,200 @@
+<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
+<HTML>
+ <HEAD>
+ <TITLE> [Mageia-dev] get-skype package for submission
+ </TITLE>
+ <LINK REL="Index" HREF="index.html" >
+ <LINK REL="made" HREF="mailto:mageia-dev%40mageia.org?Subject=Re%3A%20%5BMageia-dev%5D%20get-skype%20package%20for%20submission&In-Reply-To=%3C4DF323EA.40006%40iki.fi%3E">
+ <META NAME="robots" CONTENT="index,nofollow">
+ <META http-equiv="Content-Type" content="text/html; charset=us-ascii">
+ <LINK REL="Previous" HREF="005393.html">
+ <LINK REL="Next" HREF="005411.html">
+ </HEAD>
+ <BODY BGCOLOR="#ffffff">
+ <H1>[Mageia-dev] get-skype package for submission</H1>
+ <B>Anssi Hannula</B>
+ <A HREF="mailto:mageia-dev%40mageia.org?Subject=Re%3A%20%5BMageia-dev%5D%20get-skype%20package%20for%20submission&In-Reply-To=%3C4DF323EA.40006%40iki.fi%3E"
+ TITLE="[Mageia-dev] get-skype package for submission">anssi.hannula at iki.fi
+ </A><BR>
+ <I>Sat Jun 11 10:14:34 CEST 2011</I>
+ <P><UL>
+ <LI>Previous message: <A HREF="005393.html">[Mageia-dev] get-skype package for submission
+</A></li>
+ <LI>Next message: <A HREF="005411.html">[Mageia-dev] get-skype package for submission
+</A></li>
+ <LI> <B>Messages sorted by:</B>
+ <a href="date.html#5397">[ date ]</a>
+ <a href="thread.html#5397">[ thread ]</a>
+ <a href="subject.html#5397">[ subject ]</a>
+ <a href="author.html#5397">[ author ]</a>
+ </LI>
+ </UL>
+ <HR>
+<!--beginarticle-->
+<PRE>On 11.06.2011 03:05, Barry Jackson wrote:
+&gt;<i> On 10/06/11 18:06, Anssi Hannula wrote:
+</I>
+&gt;&gt;<i> 3. You cp files to %_datadir using a wildcard (*), but these
+</I>&gt;&gt;<i> files may not be removed on uninstallation as you only have filename
+</I>&gt;&gt;<i> lists for avatars/sounds/langs. While it may work now (I didn't
+</I>&gt;&gt;<i> test, I hope you did), this will cause unnoticed problems when the
+</I>&gt;&gt;<i> skype tarball contents change.
+</I>&gt;<i>
+</I>&gt;<i> The wildcard is used to move the remaining files which are all
+</I>&gt;<i> individually handled by touch and the dir is removed by a %ghost.
+</I>&gt;<i> I was just saving spec lines.
+</I>&gt;<i>
+</I>&gt;<i> Yes it does work fine and all files/dirs are removed on uninstall.
+</I>&gt;<i>
+</I>&gt;<i> The tarball contents can't change or the md5sum would fail :\
+</I>
+I meant that when updating the package to a new version it is easy to
+not notice some added files in the tarball, leaving the wildcard in
+place but not add respective %ghosted files.
+
+&gt;&gt;<i> 4. Provide the script/commandline used to create the filelist files.
+</I>&gt;<i>
+</I>&gt;<i> Do you mean the avatars.txt etc.
+</I>&gt;<i> It was all done semi-manually, but I will write a script if necessary.
+</I>&gt;<i> It did cross my mind, but it was quicker to just copy/paste into txt files.
+</I>&gt;<i> I was assuming I could maintain it manually.
+</I>
+Maintaining manually is fine, just add a comment beside the SourceXX
+lines to tell what to do.
+
+&gt;&gt;<i> 5. Versionize the filelist files to make sure they are renegerated
+</I>&gt;&gt;<i> when the package is updated to a new version (avatars-%version.txt)
+</I>&gt;<i>
+</I>&gt;<i> OK - again I was expecting to maintain it manually.
+</I>
+Yes, but this makes sure they are not forgotten (i.e. the pkg build
+fails completely if you forget to update them, instead of creating a
+broken package).
+
+&gt;&gt;<i> 6. Your usage of /tmp seems unsafe security-wise. What if some user
+</I>&gt;&gt;<i> has created something under /tmp/skype-%version already?
+</I>&gt;<i>
+</I>&gt;<i> Hmm.. point taken
+</I>&gt;<i>
+</I>&gt;&gt;<i> Instead use mktemp to create a temporary directory.
+</I>&gt;<i>
+</I>&gt;<i> OK ... but:
+</I>&gt;<i> I can't figure out how to get a temp filename into a %define
+</I>&gt;<i> If I use
+</I>&gt;<i> %define mytmp $(mktemp -d -q)
+</I>&gt;<i> then every reference to the %define generates a new tmp file.
+</I>&gt;<i> How can I assign the output of a command expansion to a %define so it's
+</I>&gt;<i> evaluated only once on assignment?
+</I>
+%global mytmp %(mktemp -d -q)
+
+However, that doesn't do what you want. That creates a temporary
+directory in the system where package build is not done, while you want
+to create a temporary directory in the user system. See below.
+
+&gt;<i> I can use a variable in %pre but that is invisible in %post as it's in
+</I>&gt;<i> another shell so I'm sorta stuck on that for now :(
+</I>
+Probably the easiest way is to use a fixed location under
+%{_localstatedir}/lib/%{name} for the tarball instead. Then you can use
+mktemp locally in %post.
+
+Also, is there a need to use -q for mktemp?
+
+
+--
+Anssi Hannula
+</PRE>
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+<!--endarticle-->
+ <HR>
+ <P><UL>
+ <!--threads-->
+ <LI>Previous message: <A HREF="005393.html">[Mageia-dev] get-skype package for submission
+</A></li>
+ <LI>Next message: <A HREF="005411.html">[Mageia-dev] get-skype package for submission
+</A></li>
+ <LI> <B>Messages sorted by:</B>
+ <a href="date.html#5397">[ date ]</a>
+ <a href="thread.html#5397">[ thread ]</a>
+ <a href="subject.html#5397">[ subject ]</a>
+ <a href="author.html#5397">[ author ]</a>
+ </LI>
+ </UL>
+
+<hr>
+<a href="https://www.mageia.org/mailman/listinfo/mageia-dev">More information about the Mageia-dev
+mailing list</a><br>
+</body></html>