Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(75)

Side by Side Diff: third_party/iconv/README.chromium

Issue 20220002: Add libpoppler PDF rasterizer (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: More cleanup; add iconv Windows instructions Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
(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
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698