fastsupply / growl

Automatically exported from code.google.com/p/growl
0 stars 0 forks source link

Include Growl framework dSYM with GrowlMail dSYM #201

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Encounter a crash in GrowlMail in framework code.
2. Symbolicate the crash log.
3. Examine the crash log.

What is the expected output?
The crash log identifies where in GrowlMail and in the framework the crash 
occurred, to the line of source code.

What do you see instead?
The crash log tells you where in GrowlMail the crash occurred, to the line, but 
only gives method information for the framework.

Building GrowlMail builds the framework anew, so it does not include the same 
framework that ships in the Growl SDK, so the dSYM for that framework will not 
work.

Original issue reported on code.google.com by p...@growl.info on 15 May 2011 at 12:06

GoogleCodeExporter commented 9 years ago
There are two solutions:

- Set up a folder with a ReadMe.txt file in it that instructs the person 
building GrowlMail to put a release framework and its dSYM in that folder, and 
have the GrowlMail Xcode project and Release Makefile targets use the items in 
that folder.
- Include the built framework's dSYM with GrowlMail's dSYM.

Original comment by p...@growl.info on 15 May 2011 at 12:08

GoogleCodeExporter commented 9 years ago
Changing GrowlMail over to require a dropped-in framework looks like it'd be a 
lot of work. I've just made changes that include copying the built framework's 
dSYM alongside GrowlMail's dSYM, and will be committing these changes over the 
next few minutes.

Original comment by p...@growl.info on 16 May 2011 at 2:42

GoogleCodeExporter commented 9 years ago
Fixed in [b5d642471d8c].

Original comment by p...@growl.info on 16 May 2011 at 2:48

GoogleCodeExporter commented 9 years ago

Original comment by p...@growl.info on 16 May 2011 at 2:50

GoogleCodeExporter commented 9 years ago

Original comment by chrisf.g...@gmail.com on 16 May 2011 at 2:52

GoogleCodeExporter commented 9 years ago
GrowlMail 1.2.4 is out, closing this issue.

Original comment by chrisf.g...@gmail.com on 25 May 2011 at 2:22

GoogleCodeExporter commented 9 years ago

Original comment by p...@growl.info on 29 Jul 2011 at 4:57