Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Does not work with hyperref #1

Closed
sameersingh opened this issue Nov 28, 2011 · 22 comments
Closed

Does not work with hyperref #1

sameersingh opened this issue Nov 28, 2011 · 22 comments
Assignees

Comments

@sameersingh
Copy link
Contributor

I contacted Zongfang regarding this, but this might be the right place for questions such as these.

I will the hyperref package with most of my papers to create links to sections, figures, references, etc. Unfortunately, with umthesis, I get the following errors:

Package hyperref Warning: bookmark level for unknown FrontMatter defaults to 0.

./umthsmpl.tex:178: Undefined control sequence.

\MakeUppercase ...ppercaseUnsupportedInPdfStrings

And this error appears for every \chapter, etc.

Any ideas on how to fix this? Alternatively, is there a package like hyperref that works with umthesis?

@zhangh2009
Copy link
Contributor

Hi, Sam,

This error looks the same as the error you sent to me and Zongfang

before. We have tried to fix the problem, but failed. Let me have a
discuss with Zongfang this weekend to see whether we can find a
solution to make it work.

Thanks and best regards,
Hong

On Mon, Nov 28, 2011 at 2:28 PM, sameeer
reply@reply.github.com
wrote:

I contacted Zongfang regarding this, but this might be the right place for questions such as these.

I will the hyperref package with most of my papers to create links to sections, figures, references, etc. Unfortunately, with umthesis, I get the following errors:

Package hyperref Warning: bookmark level for unknown FrontMatter defaults to 0.

./umthsmpl.tex:178: Undefined control sequence.

\MakeUppercase ...ppercaseUnsupportedInPdfStrings

And this error appears for every \chapter, etc.

Any ideas on how to fix this? Alternatively, is there a package like hyperref that works with umthesis?


Reply to this email directly or view it on GitHub:
#1

@ransford
Copy link
Member

I'm pretty sure my commit (4f326f4) fixed hyperref compatibility. @sameeer can correct me if I'm wrong.

@zhangh2009
Copy link
Contributor

Great! Thanks a lot!
--Hong

On Tue, Nov 29, 2011 at 11:30 AM, Ben Ransford
reply@reply.github.com
wrote:

I'm pretty sure my commit (4f326f4) fixed hyperref compatibility.  @sameeer can correct me if I'm wrong.


Reply to this email directly or view it on GitHub:
#1 (comment)

@sameersingh
Copy link
Contributor Author

Hi Ben,

Thanks for your check in, but its still not working for me.

Specifically, I think the umthesis version uploaded here seems much older from the latest umthesis available on Zongfang's website: http://www.cs.umass.edu/~lin/umthesis.html
Your older version doesn't work for me since it doesn't support some of the extra options. Is there a reason you went with the older version of umthesis.cls?

Further, when I included your above changes to the umthesis.cls available on Zongfang's website, it still didn't solve the problem, and throws even more obscure errors.

Let me know if you need more details.

Thanks again!

Cheers,

Sameer

@ransford ransford reopened this Nov 30, 2011
@ransford
Copy link
Member

Zongfang sent me a Subversion repository when I asked him for the most recent version. I'll compare it to the tarballed version on his umthesis page.

@ransford
Copy link
Member

The tarball did indeed contain newer stuff than the Subversion repository. Yep, I can confirm that there are some wack incompatibilities between the newer stuff and hyperref. I'll get to this at some point. Patches welcome.

@ransford
Copy link
Member

ransford commented Dec 1, 2011

Ha ha. Another UMass person had this very same problem in 2008: http://newsgroups.derkeiler.com/Archive/Comp/comp.text.tex/2008-10/msg00779.html

@ransford
Copy link
Member

ransford commented Dec 1, 2011

I've been hacking a little bit here and there, but it's obscure. In the meantime, don't use hyperref. The grad school probably hates hyperlinks anyway...

@sameersingh
Copy link
Contributor Author

Thanks for looking at it Ben. I tried playing with it too, but very soon hit against my limit on Latex knowledge.

At least we have the latest version up, and hopefully, by leaving this issue open, somebody will chime in later. Even if grad school hates links, it will definitely be useful for draft and online distribution versions, so its definitely worth pursuing the integration.

@limist
Copy link

limist commented Dec 15, 2011

Hi all: I'm able to use hyperref in my umthesis.cls document by including this code right after \begin{document}:

\pdfstringdefDisableCommands{%
\let\MakeUppercase\relax
}

The Graduate School is OK with links in the document, BUT they must be black in appearance like the rest of the body text. HTH.

@sameersingh
Copy link
Contributor Author

That works great! Can someone check this is into the sample tex file (as comments if you don't think hyperref should be enabled by default)?

This should also be included: \usepackage[colorlinks=false]{hyperref}

@zhangh2009
Copy link
Contributor

Great! Let me check it in after test.

On Tue, Dec 20, 2011 at 5:36 PM, sameeer
reply@reply.github.com
wrote:

That works great! Can someone check this is into the sample tex file (as comments if you don't think hyperref should be enabled by default)?

This should also be included: \usepackage[colorlinks=false]{hyperref}


Reply to this email directly or view it on GitHub:
#1 (comment)

@ghost ghost assigned zhangh2009 Dec 21, 2011
@zhangh2009
Copy link
Contributor

Hi, Sameeer,

I can't pass the compile with the two changes you mentioned. Could

you send me the *.cls file you are using and a *.tex file so that I
can try on my machine?

Regards,
Hong

On Tue, Dec 20, 2011 at 7:21 PM, Hong Zhang zhangh@cs.umass.edu wrote:

Great! Let me check it in after test.

On Tue, Dec 20, 2011 at 5:36 PM, sameeer
reply@reply.github.com
wrote:

That works great! Can someone check this is into the sample tex file (as comments if you don't think hyperref should be enabled by default)?

This should also be included: \usepackage[colorlinks=false]{hyperref}


Reply to this email directly or view it on GitHub:
#1 (comment)

@sameersingh
Copy link
Contributor Author

Hi Hong,

Here's the cls file and the test.tex file.

Thanks,

Sameer

On Wed, Dec 21, 2011 at 16:32, Hong Zhang <
reply@reply.github.com

wrote:

Hi, Sameeer,

I can't pass the compile with the two changes you mentioned. Could
you send me the *.cls file you are using and a *.tex file so that I
can try on my machine?

Regards,
Hong

On Tue, Dec 20, 2011 at 7:21 PM, Hong Zhang zhangh@cs.umass.edu wrote:

Great! Let me check it in after test.

On Tue, Dec 20, 2011 at 5:36 PM, sameeer
reply@reply.github.com
wrote:

That works great! Can someone check this is into the sample tex file
(as comments if you don't think hyperref should be enabled by default)?

This should also be included: \usepackage[colorlinks=false]{hyperref}


Reply to this email directly or view it on GitHub:
#1 (comment)


Reply to this email directly or view it on GitHub:
#1 (comment)

Sameer Singh
PhD Student, Computer Science
UMass, Amherst
cs.umass.edu/~sameer/

@ransford
Copy link
Member

@zhangh2009, should we close this issue? I can't tell.

@ransford
Copy link
Member

ransford commented Apr 9, 2012

Is this fixed? Should we close this issue?

@zhangh2009
Copy link
Contributor

Hi, Sameeer,

 Is this problem fixed?

--Hong

On Mon, Apr 9, 2012 at 2:46 PM, Ben Ransford
reply@reply.github.com
wrote:

Is this fixed?  Should we close this issue?


Reply to this email directly or view it on GitHub:
#1 (comment)

@sameersingh
Copy link
Contributor Author

It is fixed if I add the 2 lines by @limist above manually. @zhangh2009 wanted to check in the change after verifying it himself, I don't know if that change made it to the latest version (I don't see them in the tex or the cls file).

If those changes are committed, the issue can be considered closed.

@RicGomez
Copy link

Hi,
Apparently this problems was fixed and the case closed...however, I'm having the same problem. I tried to use the hyperref package but it results in 101 errors and no PDF

@ransford
Copy link
Member

Sorry, I can't seem to make hyperref work. @limist or @sameersingh, if you've made hyperref work for you, please submit a pull request or a patch. Adding the three lines @limist suggested -- which I also found in the newsgroup post I linked above -- after \begin{document}

\pdfstringdefDisableCommands{%
\let\MakeUppercase\relax
}

and putting these lines in umthesis.cls in the "Package loading" section

% optional: hyperref
\usepackage[colorlinks=false]{hyperref}

resulted in this fatal error:

Package hyperref Warning: Token not allowed in a PDF string (PDFDocEncoding):
(hyperref)                removing `\uppercase' on input line 187.

[6] [7] [8] [9]

Package hyperref Warning: bookmark level for unknown Chapter defaults to 0.

Runaway argument?
! Paragraph ended before \def was complete.
<to be read again> 
                   \par 
l.216 \unnumberedchapter{Introduction}

?

and I'm at an impasse until I get more time (in the middle future). Ugh, @chapter redefinitions!

@ransford
Copy link
Member

ransford commented Jul 9, 2012

I think we need to kill hyperref support, by e.g. having umthesis issue a warning if hyperref is loaded. Issue #8 comes from trying to support hyperref.

@ransford
Copy link
Member

Good news on hyperref compatibility: @sappjw contributed a patch in #11 to patch up \chapter in a saner manner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants