Closed Bug 447225 Opened 16 years ago Closed 16 years ago

Actually support --skip-calendar in comm-central's client.py

Categories

(MailNews Core :: Build Config, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Callek, Assigned: Callek)

References

Details

Attachments

(1 file)

Attached patch support --skip-calendar (deleted) — Splinter Review
We should actually support --skip-calendar (as advertised in client.py). Sorry that I missed this in my review.
Attachment #330873 - Flags: review?(kairo)
Just passing by and noted a copy & paste error in client.py that you might want to fix (or not) at the same time : 170 if not options.skip_venkman: 171 if os.path.exists(os.path.join(topsrcdir, 'mozilla', 'extensions')): 172 do_cvs_checkout(VENKMAN_DIRS, VENKMAN_CO_TAG, options.cvsroot, options.cvs, 'mozilla') 173 else: 174 print >>sys.stderr, "Warning: mozilla/extensions does not exist, ChatZilla could not be checked out."
Comment on attachment 330873 [details] [diff] [review] support --skip-calendar This looks basically good, but when changing CVS_DIRS to CALENDAR_DIRS, please also change CVS_CO_TAG to CALENDAR_CO_TAG. And please also fix that c&p error stefan pointed out.
Attachment #330873 - Flags: review?(kairo) → review+
Assignee: nobody → bugspam.Callek
pushing to ssh://hg.mozilla.org/comm-central/ searching for changes remote: adding changesets remote: adding manifests remote: adding file changes remote: added 1 changesets with 1 changes to 1 files Committed to closed tree in changeset: 2066395d65e6 a+ for closed tree per KaiRo on IRC
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Product: Thunderbird → MailNews Core
QA Contact: build-config → build-config
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: