I think most of the things you said are for the makefile right? 
Le 05-11-26 à 14:46, Youness Alaoui a écrit :

ok, here's the things I still think needs to be corrected before the release (minor things)... these things are what I noticed after extracting the deb file (make deb && dpkg -x -d dpkg amsn_0.95....) :

* I suggest we add in /usr/bin/amsn-remote and /usr/bin/amsn-remote-CLI not just /usr/bin/amsn

* there is a ~/msn/sndplay file that should be removed (Jerome ?)

You can remove it with  the makefile for Win and Linux but not yet from aMSN. This thing is used in only one place...alarm.tcl   Before, it was used to play sound everywhere in aMSN. But now I use quicktime for sound EXCEPT in alarm.tcl...why? This alarm.tcl file is such a mess, one day I took a full hour to try to figure out how to change things and..it's too hard!! I'll probably do it one day but man...who coded that?? LOL!!

* when we create the deb file, there is a /usr/share/pixmaps/msn.png that points to the compile dir where the png file was :

[EMAIL PROTECTED]:~/msn/export/pkg/usr$ ls bin/ -l

total 0

lrwxrwxrwx  1 kakaroto kakaroto 18 Nov 26 14:14 amsn -> ../share/amsn/amsn*

[EMAIL PROTECTED]:~/msn/export/pkg/usr/share$ ls -l pixmaps/

total 4

lrwxrwxrwx  1 kakaroto kakaroto 72 Nov 26 14:14 msn.png -> /home/kakaroto/msn/export/debian/amsn/usr/share/amsn/icons/48x48/msn.png


you see, the bin points to the correct amsn file, but the msn.png points elsewhere .. I think it actually points to the temporary directory when the deb file was created...


* In the lang directory, there are some files that should be removed :

LANG-HOWTO

genpage.c

missing.py

addkey.tcl

lang1.tmpl

lang2.tmpl

lang3.tmpl

langchk.sh

sortlang

complete.pl

convert.tcl

genlangfiles.c


and when I say removed, I'm talking about 'not included in the package' only!

* the AppMain.tcl file should be removed when we create a linux/windows package

I confirm

* amsn.desktop can be removed from the /usr/share/amsn/ directory...

* the sxml.tcl file is made as executable, we should remove the executable flag...

* These utils/ subdir should be removed, as they are not used yet :

pixmapoption/   pixmapbutton/  pixmapprogbar/

and maybe scalable-bg/ too, we'll need to ask Tom, but from a grep, it seems it's only used in pixmapprogbar and pixmapbutton which are removed anyways...

* the pixmaps in utils/pixmapscroll/horizontal and utils/pixmapscroll/vertical directories should be made skinable (although it might not be a good time to move all that)

* the webcamsn.tcl file in utils/webcamsn should be removed since it was only used as a test.tcl file...

So it's not used anymore? When we ask people to run the test.tcl file, it's not used?

* the utils/linux/capture/test.tcl file should be removed.

* the libtray.so should have a pkgIndex.tcl file with it and we should do a package require instead of a load... (it might not be a good time to do it also)

* fix the make deb so that it won't need to sign the deb file.. if it's needed to make an official release, then the package will have to do that additional step manually...

I'll remind you of my problem :

dpkg-deb: building package `amsn' in `../amsn_0.95cvs-20051126_i386.deb'.

make[1]: Leaving directory `/home/kakaroto/msn/export'

 dpkg-genchanges -b

dpkg-genchanges: binary-only upload - not including any source code

 signfile amsn_0.95cvs-20051126_i386.changes

gpg: WARNING: unsafe permissions on configuration file `/home/kakaroto/.gnupg/gpg.conf'

gpg: WARNING: unsafe enclosing directory permissions on configuration file `/home/kakaroto/.gnupg/gpg.conf'

gpg: skipped "Phil <[EMAIL PROTECTED]>": secret key not available

gpg: [stdin]: clearsign failed: secret key not available



The deb file seems to be created and perfectly 'healty', it just stays in the parent directory...


* the control of the debian package might need some thing :

conflicts: ccmsn

depends: tcl8.4 tk8.4 tcltls



* I'd also need someone to do the exact same thing with rpm files... I don't know how it works, but I'll see if I can extract the rpm to a directory...


* we might need to check for this kind of things also in the src package (cvs export)..


* one last thing... revert back the budy icons... please :(

hehe

Reply via email to