Dominic, you should turn /WX off globally in src/build/common.gypi.
That would simplify the patch.

I guess the incremental link failure is due to the exe size on a 32
bit OS, I'll mark it as a large binary so you can remove that step
too.

M-A

On Mon, Nov 16, 2009 at 8:58 PM, Bradley Nelson <bradnel...@google.com> wrote:
> That's good to hear!
> I see you are mainly just turning off warnings as errors in a few spots.
> Is this something that we can either gate in the build based on some flag,
> or are the warnings something that we could fix properly in the source? I'd
> love to peel off more steps from the process.
> -BradN
> On Mon, Nov 16, 2009 at 5:44 PM, Dominic Jodoin <dominic.jod...@gmail.com>
> wrote:
>>
>> Hey Brad!
>>
>> I just wanted to let you know that this is working great for me on
>> VC++ 2008 Express Edition. Thanks a lot!
>>
>> I have updated my page to reflect your change. I also added a patch
>> that modifies some GYP files in such a way that you won’t have to
>> manually “play” with the project settings in order to compile without
>> errors (with this compiler that is).
>>
>>
>> http://cotsog.wordpress.com/2009/11/08/how-to-compile-google-chrome-with-visual-c-2008-express-edition/
>>
>> -- Dominic.
>>
>> On Thu, Nov 12, 2009 at 7:51 PM, Bradley Nelson <bradnel...@google.com>
>> wrote:
>> > Ok that fix is in.
>> > You'd need to set GYP_MSVS_VERSION=2008e
>> > Let me know how that goes for you.
>> > -BradN
>> >
>> > On Wed, Nov 11, 2009 at 12:50 PM, Marc-Antoine Ruel
>> > <mar...@chromium.org>
>> > wrote:
>> >>
>> >> Updated
>> >>
>> >> http://sites.google.com/a/chromium.org/dev/developers/how-tos/build-instructions-windows
>> >> to reference your blog entry.
>> >>
>> >> I don't want to copy these instructions since it's too lengthy,
>> >> inefficient and unsupported.
>> >>
>> >> I didn't realize one could download WDK 7 without needing a MSDN
>> >> account. That's cool.
>> >>
>> >> M-A
>> >>
>> >> On Tue, Nov 10, 2009 at 10:39 PM, Dominic Jodoin
>> >> <dominic.jod...@gmail.com> wrote:
>> >> > On Tue, Nov 10, 2009 at 10:25 PM, Peter Kasting <pkast...@google.com>
>> >> > wrote:
>> >> >>
>> >> >> What do you mean?  Or to be more precise, what would considering
>> >> >> your
>> >> >> steps
>> >> >> "a valid setup to contribute" concretely result in?
>> >> >> PK
>> >> >
>> >> > I'm wondering if using a hacked ATL version 7.1 could lead to bugs
>> >> > given the product is built, I suppose, with ATL coming with Visual
>> >> > Studio 2005 or 2008 which is a different version.
>> >> >
>> >> > But what I meant was that if the steps were to be approved, I thought
>> >> > they could be included on http://dev.chromium.org.
>> >> >
>> >> > -- Dominic.
>> >> >
>> >> > --
>> >> > Chromium Developers mailing list: chromium-dev@googlegroups.com
>> >> > View archives, change email options, or unsubscribe:
>> >> >    http://groups.google.com/group/chromium-dev
>> >> >
>> >>
>> >> --
>> >> Chromium Developers mailing list: chromium-dev@googlegroups.com
>> >> View archives, change email options, or unsubscribe:
>> >>    http://groups.google.com/group/chromium-dev
>> >
>> >
>
>

-- 
Chromium Developers mailing list: chromium-dev@googlegroups.com 
View archives, change email options, or unsubscribe: 
    http://groups.google.com/group/chromium-dev

Reply via email to