yarffaJ nalA (jnala) wrote in lj_dev,
yarffaJ nalA

notes on the latest release

Ordered by decreasing importance/difficulty ratio:

  • cgi-bin/bml/scheme/global.look sets SITEROOT to be http://www.$DOMAIN and ignores $SITEROOT from ljconfig.pl. This breaks if $SITEROOT is, say, livejournal.$DOMAIN.
  • A few files have messed-up permissions, breaking the default installation:

    livejournal% find . -type f -not -perm -4 -print

    If everything should be world-readable, you could fix slips like this in release.pl.
  • ljconfig.pl: $LJDOTCOM should default to 0, %DBINFO needs a clearer explanatory comment (usual case is no slave)
  • cgi-bin/perl.core doesn't belong there. Make release.pl delete corefiles.
  • aspell is required. ljconfig.pl should allow the user to undef $SPELLER, with a comment that this is required if aspell is not installed, and update.bml should refrain from printing the spellcheck checkbox and message (line 180 of update.bml) if $SPELLER is undef.
  • images.tar.gz should have a version number.
  • Upgrading schema really needs to be done better... "if you're upgrading from X or earlier, execute blah.sql. if you're upgrading from Y or earlier, execute blugh.sql." I had no idea which ones to use, I just ran them all and prayed. :-)
  • What if perl isn't in /usr/bin? Whenever I upgrade I have to run perl -pi.bak -e 's,/usr/bin/perl,/u0/ccare/bin/perl,' $(grep -l /usr/bin/perl **/*(.x)) or something like that to change all the paths, since our Perl installation is NFS-mounted off /u0. /usr/local/bin is also a common place for Perl. Maybe an installer?

  • Post a new comment


    Anonymous comments are disabled in this journal

    default userpic

    Your reply will be screened

    Your IP address will be recorded