[Adium-devl] [Adium-svn] rev 20150 - branches/adium-1.0/Release

[Evan Schoenberg]

On Jul 9, 2007, at 1:15 AM, Juan Manuel Palacios wrote:

> -) a compiler macro that ifdef's some code in/out when building  
> beta/release;
> -) a Makefile variable that defines whether the Xcode project is  
> built in deployment-debug mode (beta true) or deployment (beta false).
>
> 	Is this correct? This is more or less what I had understood until  
> now. If so, or at least from what I understand, why have two beta  
> markers? Couldn't we do everything with a single one in the  
> Makefile from which we decide what to pass to the Adium sources as  
> the necessary compiler macro? Something like "BETA = TRUE" in the  
> makefile header and then "ifeq(BETA,TRUE) CPPFLAGS += - 
> DBETA_RELEASE_TRUE" through out the rest of the Makefile? Or is  
> this not doable when calling Xcode builds through makefiles? Or am  
> I simply understanding everything the wrong way round...? ;-)
>
> 	In any case, wouldn't it be safe to build all betas, even the  
> last, in deployment-debug mode? And then if that last one proves  
> release ready, simply remove the beta marker and rebuild, plain and  
> simple. But still, build *all* betas in beta mode?

The only difference between Deployment-Debug and Deployment in terms  
of the build process is that -DBETA_RELEASE is in the compilation  
flags in the former.  Yes, this is very much doable from the  
Makefile.  However, this makes the split between built-as-beta and  
post-processed-as-beta impossible.

The last beta is not built as-such because things may change when no  
longer building as a beta.  For example, a memory stomper bug can act  
unpredictably based upon what the surrounding code is doing; turning  
debug logs into comments (by removing the BETA_RELEASE define) is one  
such change.  Furthermore, the release process post-processes nibs  
and strings for a release build, optimizing for space; I detected a  
parse error in a .strings file earlier today ([20152]) because of  
this, an error which was undetected when building as a beta.

-Evan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://adium.im/pipermail/devel_adium.im/attachments/20070709/b9c205fb/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 186 bytes
Desc: This is a digitally signed message part
URL: <http://adium.im/pipermail/devel_adium.im/attachments/20070709/b9c205fb/attachment.sig>