Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 1 Name: libiconv | |
| 2 Short Name: libiconv | |
| 3 URL: git://git.savannah.gnu.org/libiconv.git | |
| 4 Version: v1.14 (upstream Git repo tag) | |
| 5 License: Libraries under LGPL, program under GPL | |
|
Lei Zhang
2013/08/08 21:47:56
Are you even checking out / building the "program"
ducky
2013/08/09 22:14:57
The program portion should not be built. But I thi
| |
| 6 License File: LICENSE / NOT_SHIPPED | |
|
Lei Zhang
2013/08/08 21:47:56
This should just say NOT_SHIPPED.
ducky
2013/08/09 22:14:57
Done.
| |
| 7 Security Critical: No | |
| 8 | |
| 9 Description: | |
| 10 Dependency of libpoppler | |
|
Lei Zhang
2013/08/08 21:47:56
Put in a real description of what libiconv is. It'
ducky
2013/08/09 22:14:57
Done.
Depends on how much character encodings are
| |
| 11 | |
| 12 Local Modifications: | |
| 13 No changes to files from the upstream repository. | |
| 14 Platform-specific configuration files added to third_party/iconv/config/(platfor m): | |
|
Lei Zhang
2013/08/08 21:47:56
Please try to observe the 80 col rule here too.
Lei Zhang
2013/08/08 21:47:56
s/(platform)/windows/
ducky
2013/08/09 22:14:57
???
ducky
2013/08/09 22:14:57
Done
Lei Zhang
2013/08/09 22:26:48
As in:
Platform-specific configuration files adde
ducky
2013/08/12 22:05:10
Done.
| |
| 15 config.h | |
| 16 iconv.h | |
| 17 localcharset.h | |
| 18 | |
| 19 These files were created from the header templates based on instructions from ht tp://www.codeproject.com/Articles/302012/How-to-Build-libiconv-with-Microsoft-Vi sual-Studio, then kludging it to build. The minimalistic version used here for a static build is: | |
| 20 1. Download the libiconv-1.14 tarball from http://www.gnu.org/software/libiconv/ #downloading | |
| 21 2. Copy and rename these files: | |
| 22 (libiconv-1.14)/libcharset/include/localcharset.h.build.in | |
| 23 -> third_party/iconv/config/windows/localcharset.h | |
| 24 (libiconv-1.14)/include/iconv.h.build.in | |
| 25 -> third_party/iconv/config/windows/iconv.h | |
| 26 (libiconv-1.14)/config.h.in | |
| 27 -> third_party/iconv/config/windows/config.h | |
| 28 3. Modify localcharset.h as follows: | |
|
Lei Zhang
2013/08/08 21:47:56
You may want to just include a .patch file and say
ducky
2013/08/09 22:14:57
The files are pre-included, the instructions are o
| |
| 29 Replace | |
| 30 #if @HAVE_VISIBILITY@ && BUILDING_LIBCHARSET | |
| 31 #define LIBCHARSET_DLL_EXPORTED __attribute__((__visibility__("default"))) | |
| 32 #else | |
| 33 #define LIBCHARSET_DLL_EXPORTED | |
| 34 #endif | |
| 35 with | |
| 36 #define LIBCHARSET_DLL_EXPORTED | |
| 37 4. Modify config.h as follows: | |
| 38 Delete line 30: | |
| 39 #undef EILSEQ | |
| 40 Change line 686 to: | |
| 41 #define ICONV_CONST const | |
| 42 5. Modify iconv.h as follows: | |
| 43 Replace | |
| 44 #if @HAVE_VISIBILITY@ && BUILDING_LIBICONV | |
| 45 #define LIBICONV_DLL_EXPORTED __attribute__((__visibility__("default"))) | |
| 46 #else | |
| 47 #define LIBICONV_DLL_EXPORTED | |
| 48 #endif | |
| 49 with | |
| 50 #define LIBICONV_DLL_EXPORTED | |
| 51 Delete all references to @DLL_VARIABLE@ (find and replace with nothing). | |
| 52 Find and replace these: | |
| 53 @ICONV_CONST@ -> const | |
| 54 @USE_MBSTATE_T@ -> USE_MBSTATE_T | |
| 55 @BROKEN_WCHAR_H@ -> BROKEN_WCHAR_H | |
| 56 @HAVE_WCHAR_T@ -> HAVE_WCHAR_T | |
| 57 Note: if a dynamic library is needed for whatever reason, LIBCHARSET_DLL_EXPORTE D and LIBICONV_DLL_EXPORTED needs to be defined as either __declspec(dllexport) or __declspec(dllimport). | |
| 58 | |
| 59 Autogenerated files (consistent across platforms) added to third_party/iconv/con fig: | |
| 60 aliases.h | |
| 61 canonical_dos.h | |
| 62 canonical.h | |
| 63 canonical_local.h | |
| 64 flags.h | |
| 65 | |
| 66 These files were pulled from a release tarball. | |
|
Lei Zhang
2013/08/08 21:47:56
Put this closer to line 59. At first I thought you
| |
| OLD | NEW |