On Sun Feb 18 17:13:40 2007, coke wrote: > > If I'm understanding correctly: > > > > - MANIFEST indicates which files are to be in the distribution. > > > > - svn:ignore indicates which files in a build tree are to be > > ignored as part of "svn status" and other svn commands. > > > > - MANIFEST.SKIP indicates the files in the repository (build tree?) > > that are not part of the distribution -- i.e., not part of MANIFEST.
Aye. MANIFEST.SKIP is just a safeguard to prevent extraneous files from leaking into the distribution tarball. It may contain both files committed to the repository but not included in the distribution and files generated by the build process but not included in the distribution. So, it doesn't entirely correspond to the svn:ignore property. > For the moment, disabling configure's manicheck by default would be a > good start. This isn't something we need every developer wasting > their time on given how we're using it right now; more of something > that should be handled by the release manager. > > (How many times has the build been broken by a missing MANIFEST update?) I'd agree, but then we're likely to have developers bitten by files left out of the distribution because they forgot to include them in the MANIFEST. Requiring developers to explicitly note which checked in files are excluded from the MANIFEST is good housekeeping. This falls in a category of "developer tests" that don't need to be run by the users, but do need to be run by the developers. > > I would like to get rid of MANIFEST.SKIP, however, It serves a useful purpose, as a "double accounting" check. > > and I think > > we may not be using our tool (svn) to its full advantage. > > Brainstorming a bit ... isn't it possible for us to define > > our own svn properties? (I'm not all that familiar with this > > part of svn.) If yes, then can we just have a > > "distribution:[yes|no]" svn property that indicates files to be > > part of the distribution? > > "coming soon" to svn, not available yet. Maybe someday. For now we're stuck with a manual MANIFEST. Allison
