CVS Commit: libdvdcss (gbazin)

[c***@videolan.org]

Update of /var/cvs/videolan/libdvdcss/src
Modified Files:
common.h error.c

Log Message:

* src/common.h, src/error.c: fixed the PATH_MAX breakage on win32.

[Vincent Cojot]

Post by c***@videolan.org
Update of /var/cvs/videolan/libdvdcss/src
common.h error.c
* src/common.h, src/error.c: fixed the PATH_MAX breakage on win32.

Hi everyone,

I reported a problem with PATH_MAX on Solaris a few days ago and provided
a fix. In short, it worked in all versions up to 1.2.4 and was broken in
1.2.5. The fix is to include <limits.h> on Solaris. Will there be
something done about it?

*** src/common.h.orig Wed Dec 11 10:06:07 2002
--- src/common.h Tue Mar 4 13:54:43 2003
***************
*** 44,49 ****
--- 44,53 ----
typedef signed int int32_t;
#endif

+ #if defined(__sun__)
+ # include<limits.h>
+ #endif
+
#if defined( WIN32 )

/* several type definitions */


,-*~'`^`'~*-,._.,-*~'`^`'~*-,._.,-*~'`^`'~*-,._.,-*~'`^`'~*-,._.,-*~'`^`'~*-,
Vincent S. Cojot, Computer Engineering. STEP project. _.,-*~'`^`'~*-,._.,-*~
Ecole Polytechnique de Montreal, Comite Micro-Informatique. _.,-*~'`^`'~*-,.
Linux Xview/OpenLook resources page _.,-*~'`^`'~*-,._.,-*~'`^`'~*-,._.,-*~'
http://step.polymtl.ca/~coyote _.,-*~'`^`'~*-,._ ***@step.polymtl.ca

They cannot scare me with their empty spaces
Between stars - on stars where no human race is
I have it in me so much nearer home
To scare myself with my own desert places. - Robert Frost

[H}kan Hjort]

Post by Vincent Cojot

Post by c***@videolan.org
Update of /var/cvs/videolan/libdvdcss/src
common.h error.c
* src/common.h, src/error.c: fixed the PATH_MAX breakage on win32.

Hi everyone,
I reported a problem with PATH_MAX on Solaris a few days ago and provided
a fix. In short, it worked in all versions up to 1.2.4 and was broken in
1.2.5. The fix is to include <limits.h> on Solaris. Will there be
something done about it?

It's in limits.h on Liux too.. just that the glibc headers are a bit
generous is what they export.

[Vincent Cojot]

Post by H}kan Hjort

Post by Vincent Cojot
I reported a problem with PATH_MAX on Solaris a few days ago and provided
a fix. In short, it worked in all versions up to 1.2.4 and was broken in
1.2.5. The fix is to include <limits.h> on Solaris. Will there be
something done about it?

It's in limits.h on Liux too.. just that the glibc headers are a bit
generous is what they export.

Hi Hakan,

Anyway my fix didn't fix it for libdvdcss.h. So Linux had the same problem
in 1.2.5? Well, as long as you're here, I guess it'll be fixed for
Solaris too.. :) :) :) Sorry about the excessive noise.. :)

Vincent

[Christophe Massiot]

Post by Vincent Cojot

Post by H}kan Hjort

Post by Vincent Cojot
I reported a problem with PATH_MAX on Solaris a few days ago and provided
a fix. In short, it worked in all versions up to 1.2.4 and was broken in
1.2.5. The fix is to include <limits.h> on Solaris. Will there be
something done about it?

It's in limits.h on Liux too.. just that the glibc headers are a bit
generous is what they export.

Hi Hakan,
Anyway my fix didn't fix it for libdvdcss.h. So Linux had the same problem
in 1.2.5? Well, as long as you're here, I guess it'll be fixed for
Solaris too.. :) :) :) Sorry about the excessive noise.. :)

Actually I had already included a fix about that in the CVS just after
the 1.2.5 release. Could you confirm 1.2.6-cvs fixes your problem ?

[H}kan Hjort]

Post by Christophe Massiot

Post by Vincent Cojot

Post by H}kan Hjort

Post by Vincent Cojot
I reported a problem with PATH_MAX on Solaris a few days ago and provided
a fix. In short, it worked in all versions up to 1.2.4 and was broken in
1.2.5. The fix is to include <limits.h> on Solaris. Will there be
something done about it?

It's in limits.h on Liux too.. just that the glibc headers are a bit
generous is what they export.

Hi Hakan,
Anyway my fix didn't fix it for libdvdcss.h. So Linux had the same problem
in 1.2.5? Well, as long as you're here, I guess it'll be fixed for
Solaris too.. :) :) :) Sorry about the excessive noise.. :)

Actually I had already included a fix about that in the CVS just after
the 1.2.5 release. Could you confirm 1.2.6-cvs fixes your problem ?

cvs HEAD works for me. (after deleting an old config.h file, that had
somehow ended up in src/ and was causing chaos)

[Vincent Cojot]

Post by H}kan Hjort

Post by Christophe Massiot
Actually I had already included a fix about that in the CVS just after
the 1.2.5 release. Could you confirm 1.2.6-cvs fixes your problem ?

cvs HEAD works for me. (after deleting an old config.h file, that had
somehow ended up in src/ and was causing chaos)

Hello everyone,

I didn't have time to test CVS but I noticed that 1.2.6 was released. It
compiles fine here too (Solaris 8 02/02 on Sparc). Thanks a lot..

,-*~'`^`'~*-,._.,-*~'`^`'~*-,._.,-*~'`^`'~*-,._.,-*~'`^`'~*-,._.,-*~'`^`'~*-,
Vincent S. Cojot, Computer Engineering. STEP project. _.,-*~'`^`'~*-,._.,-*~
Ecole Polytechnique de Montreal, Comite Micro-Informatique. _.,-*~'`^`'~*-,.
Linux Xview/OpenLook resources page _.,-*~'`^`'~*-,._.,-*~'`^`'~*-,._.,-*~'
http://step.polymtl.ca/~coyote _.,-*~'`^`'~*-,._ ***@step.polymtl.ca

They cannot scare me with their empty spaces
Between stars - on stars where no human race is
I have it in me so much nearer home
To scare myself with my own desert places. - Robert Frost