aMule Forum

Please login or register.

Login with username, password and session length
Advanced search  

News:

We're back! (IN POG FORM)

Author Topic: ‘wxS’ was not declared in this scope  (Read 6902 times)

Vollstrecker

  • Administrator
  • Hero Member
  • *****
  • Karma: 67
  • Offline Offline
  • Posts: 1549
  • Unofficial Debian Packager
    • http://vollstreckernet.de
‘wxS’ was not declared in this scope
« on: July 04, 2009, 10:20:29 PM »
Hi,

starting with Rev 9630 I get the following error:

i486-linux-gnu-g++ -DHAVE_CONFIG_H -I. -I../../../..  -D__DEBUG__ -D_GLIBCXX_DEBUG -D_GLIBCXX_DEBUG_PEDANTIC -DUSE_WX_EXTENSIONS -I/usr/lib/wx/include/gtk2-unicode-release-2.8 -I/usr/include/wx-2.8 -D_FILE_OFFSET_BITS=64 -D_LARGE_FILES -D__WXGTK__  -g -W -Wall -Wshadow -Wundef -ggdb -fno-inline -fmessage-length=0 -pthread   -c -o alc-ed2khash.o `test -f 'ed2khash.cpp' || echo './'`ed2khash.cpp
ed2khash.cpp:50: error: ‘wxS’ was not declared in this scope
make[5]: *** [alc-ed2khash.o] Error 1
make[5]: Leaving directory `/usr/src/amule/testbuild/src/utils/aLinkCreator/src'
make[4]: *** [all-recursive] Error 1
make[4]: Leaving directory `/usr/src/amule/testbuild/src/utils/aLinkCreator'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory `/usr/src/amule/testbuild/src/utils'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/usr/src/amule/testbuild/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/usr/src/amule/testbuild'
make: *** [all] Error 2

I crosschecked with current 9697 fresh checkout with the same result. I can see some changes for wx-2.9 in this revs, but I can't imagine that this was unrevealed for such a long time. I for myself didn't change the env between these two revs.
Logged
Homefucking is killing prostitution

wuischke

  • Developer
  • Hero Member
  • *****
  • Karma: 183
  • Offline Offline
  • Posts: 4292
Re: ‘wxS’ was not declared in this scope
« Reply #1 on: July 04, 2009, 10:47:51 PM »
You should use wxWidgets >= 2.8.8. wxS is not defined in 2.8.7.

(Alternative: Convince developers that supporting people using an outdated (2.8.8 is out for more than a year) wxWidgets version is worth it.)

Edit: >= not =>
« Last Edit: July 05, 2009, 07:59:21 AM by wuischke »
Logged

Vollstrecker

  • Administrator
  • Hero Member
  • *****
  • Karma: 67
  • Offline Offline
  • Posts: 1549
  • Unofficial Debian Packager
    • http://vollstreckernet.de
Re: ‘wxS’ was not declared in this scope
« Reply #2 on: July 04, 2009, 11:17:59 PM »
Instead of convincing devs to support debian at all, I'll just do new packages.
Logged
Homefucking is killing prostitution

Kry

  • Ex-developer
  • Retired admin
  • Hero Member
  • *****
  • Karma: -665
  • Offline Offline
  • Posts: 5795
Re: ‘wxS’ was not declared in this scope
« Reply #3 on: July 05, 2009, 07:17:16 AM »
It's more about convincing debian to support devs. 2.8.7 is awfully old.
Logged

Vollstrecker

  • Administrator
  • Hero Member
  • *****
  • Karma: 67
  • Offline Offline
  • Posts: 1549
  • Unofficial Debian Packager
    • http://vollstreckernet.de
Re: ‘wxS’ was not declared in this scope
« Reply #4 on: July 05, 2009, 11:08:07 AM »
You kniow the policy. They won't update the stable distro. And dato won't make enough pressure to get it into testing or sid.
Logged
Homefucking is killing prostitution

Stu Redman

  • Administrator
  • Hero Member
  • *****
  • Karma: 214
  • Offline Offline
  • Posts: 3739
  • Engines screaming
Re: ‘wxS’ was not declared in this scope
« Reply #5 on: July 05, 2009, 02:19:26 PM »
Reason for the change was getting errors like
Code: [Select]
alcc.h:67: error: cannot convert ‘const wchar_t*’ to ‘const char*’ in initializationwith wx 2.9
Logged
The image of mother goddess, lying dormant in the eyes of the dead, the sheaf of the corn is broken, end the harvest, throw the dead on the pyre -- Iron Maiden, Isle of Avalon

wuischke

  • Developer
  • Hero Member
  • *****
  • Karma: 183
  • Offline Offline
  • Posts: 4292
Re: ‘wxS’ was not declared in this scope
« Reply #6 on: July 05, 2009, 02:33:19 PM »
Actually we might change completely. Unless someone with better insight (Kry?) tells us why this would be a bad idea.

Quote from: Vadim Zeitlin¹
But it does make sense to use wxS() in the new code or, rather, not
use wxT/_T any more [...]
¹ Look at the google cache version for the message "wxS(): replacement of wxT/_T in the common code"
Logged

Vollstrecker

  • Administrator
  • Hero Member
  • *****
  • Karma: 67
  • Offline Offline
  • Posts: 1549
  • Unofficial Debian Packager
    • http://vollstreckernet.de
Re: ‘wxS’ was not declared in this scope
« Reply #7 on: July 05, 2009, 02:50:26 PM »
Pkg build in progress, so no problem. Maybe mention this next time somewhere.
Logged
Homefucking is killing prostitution