Bug 201021 – dev-tcltk/tclgpgme-1.0 fixes for tcl-8.5

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Max Loparev 2007-12-02 20:10:13 UTC]

I've managed to build tclgpgme with tcl8.5. Without it you'll get "unable to load shared library because of unresolved tclStubPtr" error while executing "package require tclgpgme". Because of very outdated libtool support files, i have to use autogen.sh instead of normal configure/make/make install

[Comment 1 Max Loparev 2007-12-02 20:11:09 UTC]

Created attachment 137562 [details]
new ebuild

[Comment 2 Max Loparev 2007-12-02 20:12:09 UTC]

Created attachment 137564 [details, diff]
patch for Makefile.am

[Comment 3 Max Loparev 2007-12-02 20:12:26 UTC]

Created attachment 137566 [details]
patch for configure.in

[Comment 4 Alon Bar-Lev (RETIRED) 2007-12-08 11:08:07 UTC]

Created attachment 138006 [details, diff]
tclgpgme-1.0-r3.ebuild.diff

Well, I am not tcl user... So I cannot really test this.
But please try this ebuild instead of yours.
It uses our own autotools reconfiguration, so it should be better.

[Comment 5 Alon Bar-Lev (RETIRED) 2007-12-08 11:08:43 UTC]

Created attachment 138007 [details, diff]
tclgpgme-1.0-build.patch

[Comment 6 Max Loparev 2007-12-08 11:35:19 UTC]

Created attachment 138008 [details]
checked version of ebuild 

Thank you for quick response. I've returned econf in compile section and now everything seems to be working right.

[Comment 7 Alon Bar-Lev (RETIRED) 2007-12-08 12:24:38 UTC]

Please submit only diffs not complete ebuilds.
I don't understand, what change have you made?

[Comment 8 Max Loparev 2007-12-08 13:16:31 UTC]

(In reply to comment #7)
> Please submit only diffs not complete ebuilds.
> I don't understand, what change have you made?
> 

diffs to what?
The only reason i've posted a full new ebuild is because your patch on ebuild just failing with message 

patching file tclgpgme-1.0-r3.ebuild
Hunk #2 FAILED at 19.
1 out of 2 hunks FAILED -- saving rejects to file tclgpgme-1.0-r3.ebuild.rej

so please post original on what you want to see the diff, or better, just download the previous ebuild that you had marked obsolete and run diff with my last version. Nevertheless, i've noted what i changed in comments to the new ebuild.

[Comment 9 Max Loparev 2007-12-08 13:23:08 UTC]

Created attachment 138015 [details, diff]
diff

you should get something like this :)

[Comment 10 Max Loparev 2007-12-08 13:28:48 UTC]

Created attachment 138016 [details, diff]
another diff

oops, actually you had expected this one I think

As I mentioned, to get the right diff isn't easy ;)

[Comment 11 Alon Bar-Lev (RETIRED) 2007-12-08 13:34:53 UTC]

The base is the one exists on portage.
You send me a diff of your version, this does not help us to make a progress.

Sequence:
1. mkdir /usr/local/portage/dev-tcltk
2. cd /usr/local/portage/dev-tcltk
3. cp -a /usr/portage/dev-tcltk/tclgpgme .
4. cd tclgpgme
5. patch -p0 < attachment#138006 [details, diff]
6. Test the resulting modified tclgpgme-1.0-r2.ebuild
7. If there is any problem, modify this ebuild, please attach:
diff -urNp tclgpgme-1.0-r2.ebuild /usr/portage/dev-tcltk/tclgpgme/tclgpgme-1.0-r2.ebuild

Sending only diffs enables shorter review times. Please use this method in any bug you open.

Thanks.

[Comment 12 Alon Bar-Lev (RETIRED) 2007-12-12 12:46:39 UTC]

I need to know that the exact diff of attachment#138007 [details, diff] works.

[Comment 13 Max Loparev 2007-12-13 20:38:02 UTC]

(In reply to comment #12)
> I need to know that the exact diff of attachment#138007 [details, diff] [edit] works.
> 

yes, it works

[Comment 14 Alon Bar-Lev (RETIRED) 2007-12-13 21:35:48 UTC]

Thanks!
Committed.